Skip to content

Fix @objectstack/server build configuration#1140

Merged
xuyushun441-sys merged 2 commits intomainfrom
claude/fix-ci-build-test-errors-another-one
Apr 14, 2026
Merged

Fix @objectstack/server build configuration#1140
xuyushun441-sys merged 2 commits intomainfrom
claude/fix-ci-build-test-errors-another-one

Conversation

@Claude
Copy link
Copy Markdown
Contributor

@Claude Claude AI commented Apr 14, 2026

The server build was failing because tsc attempted to type-check imported example apps which contain intentional type errors for demonstration purposes.

Changes

  • apps/server/package.json: Removed tsc from build script, keeping only objectstack compile
    • Type-checking still available via separate typecheck script
  • apps/server/tsconfig.json:
    • Changed rootDir from non-existent ./src to .
    • Narrowed include to only server-specific files: ["*.ts", "lib/**/*.ts", "server/**/*.ts", "api/**/*.ts"]
    • Added skipLibCheck: true to avoid type-checking dependencies

The server is a runtime configuration package that composes example apps at runtime - it doesn't need TypeScript compilation, only metadata compilation via objectstack compile.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
objectstack-demo Ready Ready Preview, Comment Apr 14, 2026 9:42am
objectstack-play Ready Ready Preview, Comment Apr 14, 2026 9:42am
spec Ready Ready Preview, Comment Apr 14, 2026 9:42am

Request Review

@xuyushun441-sys xuyushun441-sys marked this pull request as ready for review April 14, 2026 09:42
@xuyushun441-sys xuyushun441-sys merged commit 431747b into main Apr 14, 2026
10 of 11 checks passed
@github-actions github-actions bot added size/s dependencies Pull requests that update a dependency file tooling labels Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size/s tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants