Skip to content

feat(playwright): make demo-video call the evidence posting step #11392

Description

@kyle-sexton

Parent

Refs #8651. Evidence-verification plan: Brief and acceptance criteria at #11364 (comment); Design and Plan comments are on #11364.

Depends on

Outcome

demo-video posts evidence through /source-control:pull-request evidence when it resolves, and its own person-requested demo attach keeps its current fallback text.

Demo: run /skill-quality:check demo-video; the updated eval passes.

Plan (phase S5b)

Satisfies: AC8
Size Small. Depends on: S5a (#11391). Plugin: playwright.

  • First: grep -rn "posting.md" plugins/playwright and list consumers (known: plugins/playwright/skills/demo-video/evals/evals.json:53).
  • Replace reference/posting.md with a presence-gated call to /source-control:pull-request evidence; demo-video's own person-requested demo attach (outside evidence verification) keeps its current fallback text. Update the eval at line 53.
  • Run budget: at most 2 runs per eval case; stop on the first usage-limit error.
  • Sanity Check: test ! -f plugins/playwright/skills/demo-video/reference/posting.md || grep -q "source-control:pull-request evidence" plugins/playwright/skills/demo-video/reference/posting.md; /skill-quality:check demo-video PASS.

Design excerpt

  • source-control: /source-control:pull-request evidence posts one marker comment (local
    gh --attach; CI from source-control-babysit-merge after PASS, before enqueue); create.md's
    Visual evidence updated; playwright demo-video's reference/posting.md moves here, demo-video
    calls it.

Acceptance criteria

Brief criteria this slice satisfies:

  • AC8: A passing PR carries one comment linking redacted evidence; account and usage lines never appear in uploaded captures.

Done when:

  • Sanity Check passes: test ! -f plugins/playwright/skills/demo-video/reference/posting.md || grep -q "source-control:pull-request evidence" plugins/playwright/skills/demo-video/reference/posting.md; /skill-quality:check demo-video PASS.
  • The consumer list from grep -rn "posting.md" plugins/playwright is recorded in the PR body
  • The plugin's version and CHANGELOG are bumped per the repository's release convention
  • ci-status is green on the PR head

Held for owner decision

Dependencies only; no scope is carved out of this slice. It depends on S5a (#11391).

🤖 Generated with Claude Code

https://claude.ai/code/session_01UJUn7KoThXoYQoxfvdCVi7

Activity

  1. added
    priority: mediumReal value, no hard deadline; normal backlog flow.
    work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.
    on Oct 30, 2026
  2. github-actions commented on Oct 30, 2026

    @github-actions
    Contributor

    Proposed work class: work-class: scoped
    Basis: automated intake triage; the lanes App applied this class.

    Size: Small
    Complexity: Low
    Uncertainty: Known
    Flags: none
    Blocked by: #11391

    Understood: replace demo-video's reference/posting.md with a presence-gated call to /source-control:pull-request evidence. The person-requested demo attach keeps its fallback text, and the eval at line 53 is updated. The plan, sanity check and done-when items are exact. The work waits on #11391, which adds the evidence posting step. I found no duplicates among the open issues.

    Automated triage, applied by the lanes App.

  3. melodic-automation-lanes commented on Oct 30, 2026

    @melodic-automation-lanes
    Contributor

    Not admitted to the issue lane yet: blocker #11391 is not closed by a listed human or a merge.

  4. added
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    on Nov 1, 2026
  5. github-actions commented on Nov 1, 2026

    @github-actions
    Contributor
    No description provided.
  6. github-actions commented on Nov 1, 2026

    @github-actions
    Contributor
    No description provided.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.priority: mediumReal value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

    Type

    Fields

    No fields configured for Task.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions