Skip to content

Fix TypeScript CI failure for fumadocs workspace app#34

Merged
hotlong merged 3 commits intomainfrom
copilot/update-mobile-actions-run
Feb 10, 2026
Merged

Fix TypeScript CI failure for fumadocs workspace app#34
hotlong merged 3 commits intomainfrom
copilot/update-mobile-actions-run

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 10, 2026

CI was failing because the root TypeScript check was attempting to validate apps/docs files using the root tsconfig.json, which has incompatible path mappings (~/* vs @/*).

Changes

  • tsconfig.json: Exclude ./apps directory from root TypeScript compilation
  • .github/workflows/ci.yml: Add separate type check step for docs workspace
    • Mobile: npx tsc --noEmit (root)
    • Docs: cd apps/docs && npx tsc --noEmit

Each workspace app (fumadocs, future apps) now uses its own TypeScript configuration independently.

Original prompt

引用: https://github.com/objectstack-ai/mobile/actions/runs/21849066904/job/63051418696#step:6:1


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 10, 2026 02:31
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI changed the title [WIP] Update mobile actions run for job consistency Fix TypeScript CI failure for fumadocs workspace app Feb 10, 2026
Copilot AI requested a review from hotlong February 10, 2026 02:34
@hotlong hotlong marked this pull request as ready for review February 10, 2026 02:45
Copilot AI review requested due to automatic review settings February 10, 2026 02:45
@hotlong hotlong merged commit 25deddb into main Feb 10, 2026
4 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts TypeScript configuration and CI steps so the fumadocs workspace (apps/docs) is type-checked with its own tsconfig.json, avoiding root tsconfig.json path-mapping conflicts.

Changes:

  • Excludes ./apps from the root tsconfig.json compilation scope.
  • Splits CI TypeScript checks into separate steps for Mobile (root) and Docs (apps/docs).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
tsconfig.json Excludes workspace apps from the root TS compilation to prevent path-mapping conflicts.
.github/workflows/ci.yml Adds a dedicated docs type-check step to validate apps/docs with its own TS config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
- name: Type Check (Mobile)
run: npx tsc --noEmit
- name: Type Check (Docs)
run: cd apps/docs && npx tsc --noEmit
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The docs type-check step runs npx tsc --noEmit, which may not match the docs app’s intended type-check pipeline (types:check runs fumadocs-mdx && next typegen && tsc --noEmit). Consider invoking the workspace script (or pnpm ... exec tsc) so CI uses the docs package’s local TypeScript version and required pre-typegen steps, avoiding false negatives/positives.

Suggested change
run: cd apps/docs && npx tsc --noEmit
run: pnpm --filter ./apps/docs types:check

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants