Skip to content

Use canonical Git remote identities for sessions - #69

Merged
calvintvu merged 2 commits into
mainfrom
cvu/canonical-remote
Aug 12, 2026
Merged

Use canonical Git remote identities for sessions#69
calvintvu merged 2 commits into
mainfrom
cvu/canonical-remote

Conversation

@calvintvu

@calvintvu calvintvu commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Context

Session repository grouping currently relies on local directory names, which can collide and vary across checkout layouts. Normalize origin remotes so repositories have stable identities across machines, while preserving local-only fallbacks.

Changes

  • Normalize HTTPS, SSH, and SCP-style origin URLs to host/owner/repository.
  • Mark basename fallbacks as local-only and preserve linked worktree grouping.

Test

  • go test ./... - passed
  • go vet ./... - passed
  • npm test - passed
  • npm run build - passed
  • npm run format:check - passed

Screenshots

  • Session board - canonical remote repository label and local-only fallback
Screenshot 2026-08-12 at 11 10 46

@calvintvu calvintvu added the enhancement New feature or request label Aug 12, 2026

@luhe19001 luhe19001 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the two inline findings before merging.

Comment thread collector/internal/session/probes.go Outdated
Comment thread collector/internal/session/probes.go Outdated
@calvintvu
calvintvu requested a review from luhe19001 August 12, 2026 21:50
@calvintvu
calvintvu force-pushed the cvu/canonical-remote branch from 155bda7 to c0a2196 Compare August 12, 2026 21:51
@calvintvu
calvintvu merged commit 0791030 into main Aug 12, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants