Skip to content

docs(agents): note Studio stale-dist trap under Functional Testing#1042

Merged
christso merged 1 commit intomainfrom
docs/studio-stale-dist
Apr 11, 2026
Merged

docs(agents): note Studio stale-dist trap under Functional Testing#1042
christso merged 1 commit intomainfrom
docs/studio-stale-dist

Conversation

@christso
Copy link
Copy Markdown
Collaborator

Summary

  • Adds a paragraph to `AGENTS.md` under Functional Testing (CLI) explaining that `bun apps/cli/src/cli.ts studio` only hot-reloads the CLI / backend routes — the Studio web UI is served as a static bundle from `apps/studio/dist/` and does not rebuild on change.
  • Tells contributors (and agents) to run `cd apps/studio && bun run build` before screenshotting or driving Studio with `agent-browser`.

Why

During the post-merge UAT on #1040, my first attempt to verify the new `TagsEditor` rendered an older version of the Compare tab entirely — no mode toggle, wrong cell styling, no `PassRatePill`. Spent time convincing myself the merge had broken something before realizing `apps/studio/dist/` was from 2026-04-09 (before the work) and my source-mode invocation never touched it. The component code was correct all along; the bundle just hadn't been rebuilt.

This is the kind of thing that's easy to miss on the first run and obvious in hindsight. One paragraph in `AGENTS.md` prevents the next agent / contributor from the same confusion.

Test plan

  • `bun run lint` — clean
  • `git push` runs the prek pre-push hook (Build / Typecheck / Lint / Test / Validate eval YAML) — all passed on push
  • `AGENTS.md` renders correctly on GitHub (verified by re-opening the PR view)
  • Reviewer spot-checks the wording fits the tone of the surrounding "Functional Testing (CLI)" section

Related

🤖 Generated with Claude Code

Running `bun apps/cli/src/cli.ts studio` only live-reloads the CLI and
backend routes. The Studio web UI is served as a static bundle from
`apps/studio/dist/`, which is build output and does not recompile on
source changes. Without a manual `bun run build` in `apps/studio`,
`agentv studio` silently serves whatever JS/CSS was last built — which
may be from a different branch, before the merge you just pulled, or
simply stale.

This bit the post-merge UAT on #1040: the TagsEditor component was
correctly in the source but not in the dist, so the driven-browser
session kept rendering an older Compare tab and looked like a feature
regression. Cost ~15 minutes of confusion to diagnose.

Adds a paragraph under the existing "Functional Testing (CLI)" section
so the next agent (or human) knows to rebuild the Studio dist before
screenshotting or driving `agent-browser` against Studio.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: b3faf28
Status: ✅  Deploy successful!
Preview URL: https://72ebc01b.agentv.pages.dev
Branch Preview URL: https://docs-studio-stale-dist.agentv.pages.dev

View logs

@christso christso merged commit 8d62354 into main Apr 11, 2026
4 checks passed
@christso christso deleted the docs/studio-stale-dist branch April 11, 2026 07:37
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