chore: refresh UI dependencies and pnpm toolchain - #67
Conversation
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0deb6744f6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| }) | ||
|
|
||
| test("playground shows exact public names beside component examples", async ({ | ||
| test("shared dialog and controls remain operable across the catalog", async ({ |
There was a problem hiding this comment.
Restore catalog-wide export-name assertions
This test now stops after opening Primitives, while the traversal of Patterns, Daedalus, Creator workflow, Campaign tools, Kraken operations, Report reader, and Settings—and all corresponding data-component-name assertions—has been removed. A missing or mislabeled public-export example on those surfaces can therefore pass browser qualification; preserve the inventory coverage, potentially by splitting it into smaller tests to stay within the timeout.
AGENTS.md reference: AGENTS.md:L3-L6
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
The playground is a local harness. These hardcoded lists asserted demo registration, not package export behavior; the packed-consumer check and component map cover the public package. The dialog, keyboard, accessibility, and responsive checks remain in this test, so I am keeping the inventory deletion.
Summary
typescript-eslintdoes not support TypeScript 7 yet.setup-uvaction to v10.1.0 and apply Prettier 3.9.7 formatting.Validation
just check,just deadcode, and fresh Turbo build without the environment override: passed locally.After merge, this update supersedes Dependabot PRs #58, #57, #56, and #37.