Skip to content

fix(publish): stop screenshot spec session from stalling on oCIS checks - #97

Merged
LukasHirt merged 1 commit into
mainfrom
fix/publish-screenshot-ocis-check-prompt
Jul 24, 2026
Merged

fix(publish): stop screenshot spec session from stalling on oCIS checks#97
LukasHirt merged 1 commit into
mainfrom
fix/publish-screenshot-ocis-check-prompt

Conversation

@LukasHirt

Copy link
Copy Markdown
Owner

Summary

  • The marketplace screenshot spec prompt told Claude that oCIS is already up, but didn't forbid checking that itself — so it sometimes tried docker/curl, hit tools outside its scoped Bash allowlist (Bash(pnpm playwright test *)), got denied, and stopped to ask for permission instead of proceeding. In headless runs nobody answers, so the session stalls and the outer capture retry loop burns an attempt on a run that never executed the spec.
  • Updated prompts/marketplace-screenshots.md to explicitly forbid self-verifying oCIS readiness and to instruct that a denied tool call means "move on", not "ask/stop".

Test plan

  • Run extctl publish against an extension whose release isn't the latest and confirm the screenshot capture session no longer stalls asking for Bash permission

marketplace-screenshots.md said oCIS was already up but never told
Claude not to verify that itself. When it tried docker/curl to
double-check, the call fell outside its scoped Bash allowlist, got
denied, and it stopped to ask for permission instead of proceeding —
which stalls forever headlessly and burns an outer capture retry for
nothing. Now the prompt forbids self-checking oCIS readiness and
treats a denied tool call as "move on", not "ask/stop".

Signed-off-by: Lukas Hirt <info@hirt.cz>
@LukasHirt LukasHirt self-assigned this Jul 24, 2026
@LukasHirt
LukasHirt enabled auto-merge (squash) July 24, 2026 09:10
@LukasHirt
LukasHirt merged commit c52c8fa into main Jul 24, 2026
3 checks passed
@LukasHirt
LukasHirt deleted the fix/publish-screenshot-ocis-check-prompt branch July 24, 2026 09:11
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