Skip to content

fix: add workspace open authority diagnostics#1015

Open
EhabY wants to merge 1 commit into
mainfrom
fix/aigov-450-authority-diagnostics
Open

fix: add workspace open authority diagnostics#1015
EhabY wants to merge 1 commit into
mainfrom
fix/aigov-450-authority-diagnostics

Conversation

@EhabY

@EhabY EhabY commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add safe diagnostic logs around URI handling, remote authority creation, recent-folder matching, remote setup parsing, workspace lookup, and SSH config setup
  • stop logging full vscode:// URIs on handler failures because they may include token=...
  • add regression coverage for URI token redaction and remote authority round-tripping

Tests

  • corepack pnpm test:extension ./test/unit/util.test.ts ./test/unit/uri/uriHandler.test.ts
  • corepack pnpm typecheck
  • corepack pnpm lint
Investigation context

Linear AIGOV-450 reports failures in the VS Code Coder extension Open Workspace / URI flow with direct SSH still working. The observed invalid-authority message matches the older parser wording, while current main expects authorities in the ssh-remote+coder-vscode.<safeHostname>--<owner>--<workspace>(.<agent>) shape. This patch keeps behavior unchanged and adds focused, token-safe diagnostics so logs can correlate the dashboard URI params, generated remote authority, parsed authority, and remote setup lookup path.

Generated by Coder Agents.

@linear-code

linear-code Bot commented Jun 24, 2026

Copy link
Copy Markdown

AIGOV-450

@EhabY EhabY force-pushed the fix/aigov-450-authority-diagnostics branch 3 times, most recently from 4c3bce0 to c50c783 Compare June 24, 2026 12:50
@EhabY EhabY force-pushed the fix/aigov-450-authority-diagnostics branch from c50c783 to 5d41268 Compare June 24, 2026 12:57
@EhabY EhabY marked this pull request as ready for review June 24, 2026 14:35
@EhabY EhabY self-assigned this Jun 24, 2026
@EhabY EhabY requested a review from mtojek June 25, 2026 14:14
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