Skip to content

Add secure browser preview routing#666

Merged
chubes4 merged 1 commit into
mainfrom
issue/662-secure-public-preview
Jun 6, 2026
Merged

Add secure browser preview routing#666
chubes4 merged 1 commit into
mainfrom
issue/662-secure-public-preview

Conversation

@chubes4
Copy link
Copy Markdown
Collaborator

@chubes4 chubes4 commented Jun 5, 2026

Summary

  • Add preview-mode=local|public|secure routing metadata for browser evidence runs, with local preview remaining the default.
  • Record requested/effective preview mode, local/public/effective origins, secure-context status, and preview diagnostics in browser summaries and review artifacts.
  • Fail secure preview requests clearly when a public HTTPS preview is not configured or when the reached page is not a secure browser context.

Closes #662

Testing

  • npm run build
  • npm run recipe-preview-routing-browser-probe-smoke
  • npm run browser-probe-artifact-smoke
  • npm run browser-actions-artifact-smoke

Known issue

  • npm run artifact-contract-smoke currently fails on an existing stale metadata assertion expecting metadata.provenance.task.previewPublicUrl; current CLI metadata records preview details under metadata.provenance.task.preview.effective.publicUrl.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Implemented the preview routing/artifact metadata change and smoke coverage; Chris remains responsible for review and merge.

@chubes4 chubes4 force-pushed the issue/662-secure-public-preview branch 2 times, most recently from 3a3ebcd to 3e2353e Compare June 5, 2026 23:54
@chubes4 chubes4 force-pushed the issue/662-secure-public-preview branch from 3e2353e to 4c3b91c Compare June 6, 2026 00:04
@chubes4 chubes4 merged commit 60021d5 into main Jun 6, 2026
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.

Add secure public preview support for browser evidence runs

1 participant