Skip to content

chore: keep npm package documentation minimal - #9

Merged
254808127 merged 2 commits into
mainfrom
docs/dsh-install-prompt
Aug 19, 2026
Merged

chore: keep npm package documentation minimal#9
254808127 merged 2 commits into
mainfrom
docs/dsh-install-prompt

Conversation

@254808127

Copy link
Copy Markdown
Collaborator

Summary

  • remove internal docs/ materials from the repository tree
  • ignore docs/ in Git and npm packaging
  • keep only README.md as package documentation
  • remove Linux-only content from the current user-facing README
  • update the preview status after Windows x64 and macOS ARM64 E2E validation

Validation

  • npm run check
  • npm run package:verify
  • npm payload contains 19 files and only README.md as Markdown
  • git diff --check

Linux support remains deferred and is not added by this change.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Claude finished @254808127's task in 1m 4s —— View job


Code Review

  • Read diff and analyze for bugs and security issues

One bug found.

README.md line ~51 contains a truncated commit hash:

`native-source.json` pins `browser-cli` v1.1.13 at commit `3af544780365309fe97d51b631070e7ca73762`.

The hash is 38 hex characters; a valid SHA-1 is 40. It is missing ae after 309fe. The authoritative value in native-source.json is:

3af544780365309feae97d51b631070e7ca73762

Fix this →

No other bugs or security issues found. The .gitignore/.npmignore/package.json changes are consistent and correct.


@github-actions github-actions 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.

Automated Claude review completed. Review run: https://github.com/lexmount/dsh-browser/actions/runs/32232118396. Please check Claude's inline feedback and summary comment for details.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Claude finished @254808127's task in 1m 0s —— View job


Code Review

  • Read diff and analyze for bugs and security issues

No bugs or security issues found.

The previously flagged truncated commit hash (3af544780365309fe97d51b631070e7ca73762, 38 chars) has been corrected. README.md line 31 now reads 3af544780365309feae97d51b631070e7ca73762 (40 chars), matching native-source.json exactly.

All other changes are consistent:

  • .gitignore and .npmignore correctly exclude docs/
  • package.json files array removal of "docs/*.md" matches the deleted directory
  • README platform table and cache table removals match the stated Linux-deferred scope

@github-actions github-actions 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.

Automated Claude review completed. Review run: https://github.com/lexmount/dsh-browser/actions/runs/32232468994. Please check Claude's inline feedback and summary comment for details.

@254808127
254808127 merged commit 9b93a8b into main Aug 19, 2026
3 checks passed
@254808127
254808127 deleted the docs/dsh-install-prompt branch August 19, 2026 09:05
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