Skip to content

chore: refresh UI dependencies and pnpm toolchain - #67

Merged
JJLiebig merged 4 commits into
mainfrom
chore/refresh-dependencies
Sep 24, 2026
Merged

JJLiebig merged 4 commits into
mainfrom
chore/refresh-dependencies

Conversation

@JJLiebig

@JJLiebig JJLiebig commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Pin pnpm 12.6.0 across the workspace, documentation, and CI; allow the current Chocolatey pnpm to run the pinned workspace through Turbo.
  • Refresh direct dependencies to the newest versions permitted by the workspace's seven-day release policy. TypeScript remains at 6.0.3 because typescript-eslint does not support TypeScript 7 yet.
  • Update the pinned setup-uv action to v10.1.0 and apply Prettier 3.9.7 formatting.
  • Start the Playwright preview server directly so it shuts down cleanly on Ubuntu. Remove hardcoded catalog inventory assertions while retaining behavior, accessibility, and responsive checks.

Validation

  • Frozen install with pnpm 12.6.0, just check, just deadcode, and fresh Turbo build without the environment override: passed locally.
  • Focused catalog browser tests: 5/5 passed locally.
  • Qualification on current head: Quality, Correctness, and all 48 browser tests passed.
  • Local Review Suite signoff: clean.

After merge, this update supersedes Dependabot PRs #58, #57, #56, and #37.

@JJLiebig

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-24T00:15:53.272665Z 0deb674 Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: c3023d49be

ℹ️ 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".

@JJLiebig

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment thread tests/e2e/catalog.spec.ts
})

test("playground shows exact public names beside component examples", async ({
test("shared dialog and controls remain operable across the catalog", async ({

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

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.

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.

1 participant