Skip to content

🔐 Add secure project link credentials#271

Merged
Robdel12 merged 2 commits into
rd/visual-context-cli-contextfrom
rd/visual-context-cli
May 21, 2026
Merged

🔐 Add secure project link credentials#271
Robdel12 merged 2 commits into
rd/visual-context-cli-contextfrom
rd/visual-context-cli

Conversation

@Robdel12
Copy link
Copy Markdown
Contributor

Summary

This PR layers the secure cloud auth flow on top of the CLI context work from #269.

  • Separates user login from project upload credentials
  • Adds vizzly project link <org>/<project> for minting project-scoped upload credentials
  • Stores linked project tokens in macOS Keychain when available, with config-file fallback
  • Uses linked project tokens for cloud uploads while keeping user JWTs for review actions
  • Prevents failed project-token requests from consuming user refresh tokens
  • Updates review commands so approval/comment actions require user auth instead of project tokens
  • Adds credential-store coverage for Keychain/file fallback behavior

Testing

  • node --test tests/commands/project.test.js tests/utils/config-loader.test.js tests/project/core.test.js tests/project/operations.test.js tests/commands/login.test.js tests/commands/review.test.js tests/api/client.test.js
  • node --test tests/utils/project-link-store.test.js tests/commands/project.test.js tests/utils/config-loader.test.js tests/api/client.test.js tests/commands/review.test.js
  • npm run lint
  • npm run build
  • git diff --check

Robdel12 added 2 commits May 20, 2026 22:55
Separate user login from project upload credentials, store linked project tokens via keychain when available, add vizzly project link, and keep review commands on user auth.
@Robdel12 Robdel12 merged commit b9455fb into rd/visual-context-cli-context May 21, 2026
@Robdel12 Robdel12 deleted the rd/visual-context-cli branch May 21, 2026 04:03
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