Skip to content

docs: document minimal web support#831

Open
thymikee wants to merge 2 commits into
mainfrom
codex/docs-web-minimal-support
Open

docs: document minimal web support#831
thymikee wants to merge 2 commits into
mainfrom
codex/docs-web-minimal-support

Conversation

@thymikee

@thymikee thymikee commented Jun 19, 2026

Copy link
Copy Markdown
Member

Summary

Document the managed web backend as the supported browser path across README, CLI help, website command/client/MCP docs, package metadata, and SkillGym guidance.

Closes #822

Touched files: 9. Scope stays in docs, help, package metadata, and command-planning guidance.

Validation

pnpm exec vitest run src/utils/tests/args.test.ts
pnpm check:tooling

SkillGym focused validation was blocked in this Codex sandbox because external runners would receive branch-local workspace content; run pnpm test:skillgym:case web-first-use-runs-managed-setup from a normal authenticated local shell if needed.

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://callstack.github.io/agent-device/pr-preview/pr-831/

Built to branch gh-pages at 2026-06-20 10:24 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.3 MB 1.3 MB +4.0 kB
JS gzip 422.3 kB 423.3 kB +999 B
npm tarball 557.3 kB 558.7 kB +1.4 kB
npm unpacked 1.9 MB 1.9 MB +5.8 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 29.1 ms 28.7 ms -0.3 ms
CLI --help 46.8 ms 46.2 ms -0.6 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/cli-help.js +4.0 kB +999 B

@thymikee

Copy link
Copy Markdown
Member Author

The docs/help diff looks aligned with #822, but I would not call this merge-ready until the new SkillGym case has either run or been explicitly accepted as unverified.

This PR adds web-minimal-browser-loop in test/skillgym/suites/agent-device-smoke-suite.ts, and the repo testing matrix requires:

pnpm test:skillgym:case web-minimal-browser-loop

The PR body currently says that command could not run in the sandbox because the runner would send unpublished workspace content to external services. That is a valid blocker, but it should stay as residual risk unless someone runs the focused case in an approved environment or maintainers explicitly decide to waive it for this docs/help change.

@thymikee thymikee force-pushed the codex/docs-web-minimal-support branch from 4275111 to ba6623f Compare June 20, 2026 10:24
@thymikee

Copy link
Copy Markdown
Member Author

After #833 merged into main, GitHub now reports this PR as not merge-ready (mergeStateStatus: UNSTABLE). The prior checks are green, but they ran before the current main; please update/rebase this branch onto latest main, resolve any overlap in the web docs/help files, and rerun the required validation before merging.

@thymikee thymikee left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Docs read clearly and stay consistent with the new managed web backend story across README, CLI help, website docs, args tests, and the SkillGym case. One small fix inline: website/docs/docs/commands.md now lists agent-device help web twice. Everything else looks good.


Generated by Claude Code

agent-device help react-devtools
agent-device help remote
agent-device help web
agent-device help macos

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This adds a second agent-device help web entry — the list already contains one a few lines down (right after agent-device help macos), so the rendered help-topic list now shows help web twice. This new placement (between help remote and help macos) is the one that matches the canonical ordering in src/utils/cli-help.ts (AGENT_WORKFLOWS), so the stale duplicate after help macos should be removed instead.


Generated by Claude Code

@thymikee

Copy link
Copy Markdown
Member Author

Worker steering from latest review sweep (since 2026-06-19): please resolve the unresolved inline review thread in website/docs/docs/commands.md by removing the stale duplicate agent-device help web entry after help macos; keep the new placement between help remote and help macos because it matches AGENT_WORKFLOWS. Also either run pnpm test:skillgym:case web-minimal-browser-loop in an approved environment or explicitly keep/confirm that validation as residual risk. Current branch/check state is clean and green, so no rebase is requested unless main moves again.

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.

docs: document minimal web support

1 participant