Skip to content

shares: git-native sharing — full refactor - #97

Merged
mitschabaude merged 16 commits into
mainfrom
share-viewer-update
Mar 3, 2026
Merged

shares: git-native sharing — full refactor#97
mitschabaude merged 16 commits into
mainfrom
share-viewer-update

Conversation

@mitschabaude

@mitschabaude mitschabaude commented Mar 3, 2026

Copy link
Copy Markdown
Member

Completes the git-native sharing refactor. The server endpoints, repo-id registration, and plain-text .shares/ format were landed in earlier PRs. This PR wires up the frontend and removes the legacy system.

What's new here

Frontend — git-native share creation

  • Share creation/revocation via GitHub Contents API (pure git, no server session needed)
  • Share state discovered through the existing bidirectional sync — no separate cache
  • Share viewer updated for Tier 1 (/s/<owner>/<repo>/<shareId>) and Tier 2 (/s/<segment>) URL formats

Legacy removal

  • Removed getShareLinkForNote/createShareLink/revokeShareLink from backend.ts
  • Removed /v1/shares and /v1/share-links/:id endpoints, sharing.ts, share-store.ts

Text file support

  • kindFromPath consolidated as single source of truth for file kind inference
  • No-extension files (Makefile, .gitignore, .shares/*) → 'text'; known extensions allowlisted; unknown → null (skip sync)
  • TextEditor component for viewing/editing plain-text files (full-page height)

Tooling

  • skills/vibenote/SKILL.md updated with git-native sharing workflow
  • AGENTS.md documents e2e browser testing with agent-browser
  • .browser-auth.json gitignored for persistent browser auth state

@vercel

vercel Bot commented Mar 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
vibenote Ready Ready Preview, Comment Mar 3, 2026 1:47pm

…ension allowlist; no-extension files are text
- Delete server/src/sharing.ts and share-store.ts (old server-stored shares)
- Remove SHARE_STORE_FILE from env.ts
- Remove sharingEndpoints wiring from index.ts
- Remove old getShareLinkForNote/createShareLink/revokeShareLink from backend.ts
- Clean up dead share mocks from data.test.ts
- Update skills/vibenote/SKILL.md for git-native sharing workflow
@mitschabaude mitschabaude changed the title shares: update share viewer for git-native URL formats shares: git-native sharing — full refactor Mar 3, 2026
@mitschabaude
mitschabaude merged commit 645d2ea into main Mar 3, 2026
3 checks passed
@mitschabaude
mitschabaude deleted the share-viewer-update branch March 3, 2026 13:50
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