Update deno.json and README.md for improved dependency management and…#170
Update deno.json and README.md for improved dependency management and…#170florianbgt merged 1 commit intomainfrom
Conversation
… documentation - Added new dependencies to deno.json, including various Radix UI components and Tailwind CSS. - Updated README.md to enhance project description, features, installation instructions, and CLI commands. - Refactored the structure of the README for better clarity and added new sections for quick start and supported languages. - Adjusted the CLI commands to reflect the new structure and functionality.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7faeb80. Configure here.
| if (props.name.length > maxChar) { | ||
| const displayedName = props.truncateBefore | ||
| ? "..." + props.name.slice(0, maxChar) | ||
| : props.name.slice(0, maxChar) + "..."; |
There was a problem hiding this comment.
truncateBefore shows wrong end of string
Medium Severity
When truncateBefore is true, the intent is to truncate the beginning of the string and show the end (e.g., ...authenticate (function)). However, props.name.slice(0, maxChar) still takes the first maxChar characters, so the result is "..." + first N chars — identical to the non-truncated case but with the ellipsis on the wrong side. It needs props.name.slice(-maxChar) to take the last maxChar characters. This affects three call sites displaying symbol and file names in the viewer UI.
Reviewed by Cursor Bugbot for commit 7faeb80. Configure here.
| commitShaDate: string; | ||
| createdAt: string; | ||
| manifest: DependencyManifest; | ||
| } |
There was a problem hiding this comment.
ManifestEnvelope and constants duplicated across three handlers
Low Severity
The NAPI_DIR and MANIFESTS_DIR constants and the ManifestEnvelope interface are independently defined in extract/index.ts, generate/index.ts, and view/index.ts. If the directory name or envelope shape changes, all three must be updated in sync. Extracting these into a shared module would reduce maintenance burden and the risk of inconsistencies.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 7faeb80. Configure here.
1 participant
… documentation
Type of change
not work as expected)
Description
Please include a summary of the changes and the related issue. Explain the
motivation behind this change.
Related Issue
Issue Number: #<issue_number>
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Checklist
Note
Medium Risk
Shifts core CLI flows (
generate,extract) from API-backed/authenticated behavior to local.napi/manifestsfiles and introduces a new embedded viewer build/serve pipeline, so regressions could impact primary workflows and packaging across platforms.Overview
napiis refocused to be fully offline:generatenow writes a manifest envelope to.napi/manifests/{timestamp}-{sha}.json, andextractreads from local manifests (defaulting to the latest) instead of fetching from the NanoAPI API.Adds a new
napi viewcommand that serves the bundled React viewer plus local API endpoints (/api/manifests,/api/manifests/:id,/api/manifests/:id/audit) and opens the browser automatically; audit thresholds are added to.napircand anauditManifestis generated server-side.Removes API/auth plumbing (
ApiService,login,manifestcommand, auth middleware) and updates build/dev tooling (deno.jsontasks to build/includeviewer/dist, new viewer dependencies, CI step labels), plus documentation and example.napircupdates (newauditconfig and local manifest storage guidance).Reviewed by Cursor Bugbot for commit 7faeb80. Bugbot is set up for automated code reviews on this repo. Configure here.