Skip to content

Fix CI test failures: syntax error and undefined schema references#33

Merged
huangyiirene merged 2 commits intocopilot/add-international-homepagefrom
copilot/fix-action-step-issue
Jan 20, 2026
Merged

Fix CI test failures: syntax error and undefined schema references#33
huangyiirene merged 2 commits intocopilot/add-international-homepagefrom
copilot/fix-action-step-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 20, 2026

CI tests were failing due to a syntax error in field tests and references to non-existent MenuItem schemas.

Changes

  • field.test.ts: Removed duplicate closing brace at line 151

    -    });
         });
       });
  • manifest.test.ts: Removed MenuItem test suite that referenced undefined MenuItemSchema and MenuItem types from manifest.zod.ts

    • Navigation items are properly defined in app.zod.ts via NavigationItemSchema
    • Removed 26 lines of invalid test code

Impact

All 273 tests now pass. Build completes successfully.

Original prompt

引用: https://github.com/objectstack-ai/spec/actions/runs/21158118783/job/60847665392#step:8:1


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

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 20, 2026

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

Project Deployment Review Updated (UTC)
spec Ready Ready Preview, Comment Jan 20, 2026 3:38am

Request Review

Co-authored-by: huangyiirene <7665279+huangyiirene@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix action step issue in GitHub workflow Fix CI test failures: syntax error and undefined schema references Jan 20, 2026
Copilot AI requested a review from huangyiirene January 20, 2026 03:39
@huangyiirene huangyiirene marked this pull request as ready for review January 20, 2026 03:40
@huangyiirene huangyiirene merged commit 716b5e9 into copilot/add-international-homepage Jan 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants