Skip to content

Update README project structure - #28

Merged
cohogan merged 1 commit into
mainfrom
f/readme-accuracy
Aug 7, 2026
Merged

Update README project structure#28
cohogan merged 1 commit into
mainfrom
f/readme-accuracy

Conversation

@cohogan

@cohogan cohogan commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Audited the README against the current codebase. Almost everything was accurate; two spots had drifted:

  • src-tauri/src/ structure was missing diagnostics.rs (network diagnostics) and http_client.rs (shared HTTP client)
  • src/hooks/ listing was missing use-session-context

Verified accurate as-is: features list (notify watcher, CRC32C, 30s heartbeat, glob ignore patterns, updater), tech stack versions (React 19, Vite 8, Tailwind v4), dev port 1420, pnpm types endpoint, pairing flow endpoints (get-code, poll-pairing), CI/CD description, and VITE_SERVER_URL.

Also bumps version to 0.2.1 per repo convention. Note: merging will draft an app-v0.2.1 release — leave it unpublished until the 0.2.0 auto-update test is done, since the updater only sees published releases.

@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Claude finished @cohogan's task in 49s —— View job


Code Review: PR #28

  • Review diff and verify changes
  • Verify README accuracy against codebase
  • Check version bump consistency
  • Post review findings

Looks good. Clean, accurate PR — no issues found.

README additions verified against codebase:

  • diagnostics.rs exists and does network diagnostics (connectivity checks) — description accurate
  • http_client.rs exists and provides a shared reqwest::Client + response helpers — description accurate
  • use-session-context.tsx exists in src/hooks/ — confirmed
  • main.rs is intentionally omitted from the listing (standard Tauri entry point) — reasonable

Version bump (0.2.0 → 0.2.1) consistent across all three locations:

  • package.json:4
  • src-tauri/Cargo.toml:3
  • src-tauri/tauri.conf.json:4
  • src-tauri/Cargo.lock updated to match ✓

One note from the PR description worth flagging for the merge: the author correctly notes that merging will draft an app-v0.2.1 release — leave it unpublished until the 0.2.0 auto-update test completes.

No changes requested. LGTM.

@cohogan
cohogan merged commit 70d1b99 into main Aug 7, 2026
1 check passed
@cohogan
cohogan deleted the f/readme-accuracy branch August 7, 2026 04:49
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