feat: implement Dashboard component with controlled/uncontrolled navi… - #648
Merged
k-deejah merged 1 commit intoSep 3, 2026
Merged
Conversation
…gation and per-screen error boundaries
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.
Summary
Closes #569
package.jsonwas missingmain,module,exports, andfilesfieldsneeded for npm distribution, and
vite.config.tsonly had an app build(index.html entry), not a library build mode.
Changes
src/components/index.ts,outputting ESM and CJS bundles plus TypeScript declarations to
dist/package.json: addedmain,module,types,exportsmap, andfiles: ["dist", "README.md", "CHANGELOG.md"]Note on related issue #568
Issue #568 in this repo has near-identical scope. [FILL IN: state here whether
#568 was merged, still open, or unstarted at the time of this PR, and how this
PR relates to it — reviewer will want this context.]
Verification
npm test— 83 test files / 1,352 tests passingnpm run typecheck— 0 errorsnpm run test:exports— passingnpm run build— dist/index.js, dist/index.cjs, dist/index.d.ts,dist/style.css all generated
npm pack --dry-runtarball contents — not yet confirmed against theacceptance criteria (dist/, README.md, CHANGELOG.md only)
Note on diff scope
This branch's working history touched some files outside the immediate
package.json/vite-config scope during investigation
(SorokitProvider.tsx/Dashboard.tsx area). [FILL IN: confirm here whether the
final diff actually includes changes to those files, and if so, why they were
necessary for this fix — if they're not needed, please remove them from the
diff before merging.]