Repository navigation
Part A: Release and packaging fixes - #41
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The changesets fixed group keeps @javaabu/onu and @javaabu/onu-mcp together, and scripts/sync-plugin-version.mjs keeps the plugin manifest in step; the release workflow fails if it differs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The @font-face rules move to fonts-thaana.css, imported by tailwind.css, styles.css and fonts.css. New budget: fonts-thaana.css 400 B (327 B measured). tailwind.css stays within 8.5 kB (7.97 kB) and styles.css within 4.5 kB (4.38 kB). fonts.css drops to 400 B now the font faces moved out (306 B). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Both share React context with the app, so a second copy silently disconnects them. @tanstack/react-query is a required peer, not optional as first planned: the core entry imports ApiSelect statically, so an app without the package would fail to build. The fallback QueryClient moves from a module singleton (shared across SSR requests) to each OnuProvider, or the component when there is no provider. getOnuQueryClient is gone, and invalidateApiOptions takes the client first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Nothing in src, web or templates used it; every Onu select is built on cmdk. The primitives budget drops from 165 kB to 130 kB (124.25 kB measured). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
pnpm 12 keeps the version from packageManager in a section of pnpm-lock.yaml, and a frozen install fails if it is missing. Moving to 12.16.1 in 15ccf53 changed package.json only, so every CI job has failed at pnpm install since. It didn't show locally because an older global pnpm that switches itself to 12.16.1 doesn't check this. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
next 16.4.3 (critical RCE and high DoS, templates/nextjs). shadcn 4.23.0, a dev dependency, which moves its zod to 4.6.7. In the lockfile, within existing ranges: body-parser 2.4.0, qs 6.17.1, fast-uri 3.1.9, ip-address 10.7.5, undici 7.30.1 and 8.12.3, systeminformation 5.33.19. pnpm audit --prod is clean. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part A: release and packaging.
Stacked, merge in order, and do not squash: the branches go
plan/a-packaging→plan/b-bugs→plan/c-api→plan/d-ci→plan/e-docs, each built on the one before. All targetdevas the plan asks, so until the earlier PRs merge this diff includes them. Review it commit by commit (one commit per plan step).Steps
@javaabu/onuand@javaabu/onu-mcp;scripts/sync-plugin-version.mjskeeps the plugin manifest in step, checked in the release workflow.fonts-thaana.css, imported bytailwind.css,styles.cssandfonts.css, so Thaana works without the font pack. Verified in thebuild:weboutput.react-hook-formand@tanstack/react-querybecome peers; the fallbackQueryClientbelongs to eachOnuProvider.Comboboxprimitive and@base-ui/react.Deviation from the plan
@tanstack/react-queryis a required peer, not optional. The core entry importsApiSelectstatically, so an optional peer would break the build of any app without it. This is the same reasoning the plan gives for keepingreact-aria-componentsa dependency.Renamed or removed APIs
getOnuQueryClient()useOnuQueryClient()invalidateApiOptions(url, client?)invalidateApiOptions(client, url?)Combobox*in/primitives@tiptap/extension-link,@tiptap/extension-drag-handle-react,@tiptap/extension-code-block-lowlight,lowlightSize limits at this tip
Budgets changed:
primitives165 → 130 kB,fonts.css600 → 400 B, newfonts-thaana.css400 B.Checks
At this tip: typecheck, 2591 unit tests,
sizepass. Lint, format and docs check pass at this tip; A3 was committed without the per-commit script.build:webpassed at A6.🤖 Generated with Claude Code