Skip to content

docs(deploy): add Agent Browser post-deploy verification template#34

Merged
Abdulmuiz44 merged 1 commit into
mainfrom
feat/codra-deploy-agent-browser-template
Jun 10, 2026
Merged

docs(deploy): add Agent Browser post-deploy verification template#34
Abdulmuiz44 merged 1 commit into
mainfrom
feat/codra-deploy-agent-browser-template

Conversation

@Abdulmuiz44

Copy link
Copy Markdown
Collaborator
  • Adds Codra Deploy integration docs for Agent Browser post-deploy smoke checks.
  • Adds GitHub Actions example using externally verified talocode/agent-browser@v0.
  • Documents why HTTP 200 is not enough and the future Codra report → explain → fix loop.
  • Adds roadmap note for future codra deploy verify <url> / codra browser check <url> without implementing commands yet.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0ac471681c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

with:
url: ${{ steps.deploy.outputs.url }}
screenshot-out: agent-browser-screenshot.png
vision: "false"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Enable vision before advertising blank-page failures

This example enables fail-on-blank while setting vision: "false", but the Agent Browser v0 action only evaluates blank-page failures when vision is enabled (vision && failOnBlank). As written, users who copy this recommended smoke check will not get the documented blank-page vision failure signal; either enable vision in the example or avoid claiming this workflow fails on blank-page checks.

Useful? React with 👍 / 👎.

## Safety notes

- Agent Browser only allows public `http://` and `https://` URLs by default.
- Private network and localhost targets are blocked unless explicitly overridden for local development.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Don't overstate private-network blocking

For workflows where the checked URL can be influenced by deploy output or PR configuration, this safety guarantee is broader than Agent Browser v0 actually enforces: its URL safety code rejects localhost and only a subset of literal private IP ranges, but does not resolve hostnames and does not block literal link-local addresses such as 169.254.x.x. Please narrow this sentence to the actual restrictions or document that callers must still trust/validate the target URL.

Useful? React with 👍 / 👎.

@Abdulmuiz44 Abdulmuiz44 merged commit fbc5fc1 into main Jun 10, 2026
1 check passed
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