Skip to content

Address pre-publish review findings for react-ui - #41

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-8200-address-pre-publish-review-findings-for-react-ui
Sep 21, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-8200-address-pre-publish-review-findings-for-react-ui

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Correct the public module count to the generator output (197: 152 UI, 5 blocks, 16 hooks, 24 lib) in README and CHANGELOG.
  • Graduate CHANGELOG 0.1.0 from Unreleased to a dated release section (2026-09-17); Unreleased now tracks only the post-0.1.0 AuthLayout breaking change.
  • Collapse the git-install / prepare / trustedDependencies docs to a contributor fallback; npm registry install is primary.
  • Complete the peer-install commands with @radix-ui/react-dropdown-menu and @radix-ui/react-tooltip (matching peerDependencies); same fix in the CHANGELOG peer list.
  • Set package.json author to Sawyer Cutler sawyer@dirtroad.dev.

Verification

  • bun run typecheck: pass (tsc --noEmit, exit 0)
  • bun run lint: pass (eslint ., exit 0)
  • bun run dep-guard: pass (clean, exit 0)
  • bun test: pass (336 pass, 0 fail across 49 files)
  • npm pack --dry-run: pass (exit 0, 405 files, 323.9 kB; prepack full build incl. contrast gate ran clean)

Fixes CL-8200

@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Review

Verdict: approve (docs). CHANGELOG, README, package.json author. No CI (main has no workflows). No prior reviews. README/author will conflict with #42/#48.

npm view @corbits/react-ui 404s; do not advertise a registry tarball as
the primary install.
@TheGreatAxios
TheGreatAxios merged commit 7159dee into main Sep 21, 2026
@TheGreatAxios
TheGreatAxios deleted the cl-8200-address-pre-publish-review-findings-for-react-ui branch September 21, 2026 04:17
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