Skip to content

Remove stale CUA smoke-test mandate from AGENTS.md #239

Closed

Description

@dzianisv

Context

The CUA smoke test workflow .github/workflows/cua-smoke.yml was deleted from main on 2026-11-13 (commit e844493, owner order: no credits to run it). It is not a required check.

AGENTS.md still mandates it. Two blocks are stale:

  1. ### When to run CUA test — mandates running the CUA smoke test before any merge to main or release, and says "If the CUA test fails, do NOT merge or release until fixed."
  2. ### CI — states the workflow .github/workflows/cua-smoke.yml exists and runs on push to main and on v* tags. The file is gone, so both claims are false.

Scope

  • Remove the MANDATORY mandate block in ### When to run CUA test, replaced by a short note that the test was removed 2026-11-13 and is no longer required before merge or release.
  • Remove the ### CI subsection describing the deleted workflow.
  • Keep the rest of the CUA section intact: scripts/android-cua-smoke.py still exists in the repo and can be run locally for UI debugging.
  • Touch nothing else in AGENTS.md. No code changes, no test changes. Do not change the applicationId.

Acceptance criteria

  1. AGENTS.md contains no mandate to run the CUA smoke test before merge or release.
  2. AGENTS.md contains no claim that cua-smoke.yml exists or runs in CI.
  3. The diff touches only the mandate block and the CI subsection; the rest of AGENTS.md is unchanged.
  4. The PR passes CI.
  5. No Design/PRD update needed (doc fix — recorded on this issue at Stage 5.5).

Activity

  1. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Claim — Ember (Muse), owning lead for this task. Branch: dzianisv/docs/agents-cua-mandate-239. Scope: remove the stale CUA smoke-test mandate block and the CI subsection describing the deleted cua-smoke.yml workflow from AGENTS.md; script usage docs stay; no code or test changes.

  2. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Context pack — Stage 0, lead Ember (Muse).

    • Current stage: Stage 0 claim posted; next: Stage 3 implementation.
    • Verified base: main HEAD cad292a (PR fix(chat): auto-scroll transcript to newest message (#155) #238 merge, 2026-11-13). Workflow .github/workflows/cua-smoke.yml absent from the tree at that SHA; scripts/android-cua-smoke.py present.
    • Prior decision: CUA smoke test removed 2026-11-13 on owner order (no credits), commit e844493. The mandate in AGENTS.md is stale since that commit.
    • Nobody-else check (2026-11-13): no open issue and no open PR touches AGENTS.md or the CUA mandate; no AGENTS.md commits in the last 48h.
    • Lane: doc fix, no design stage. No Google AI Studio, no test changes. PR must pass CI before merge.
    • Acceptance: AGENTS.md has no CUA mandate before merge/release; no claim cua-smoke.yml exists or runs in CI; diff touches only the mandate block + CI subsection; PR CI green; no Design/PRD update needed (recorded at Stage 5.5).
    • Next action: Stage 3 implementation subagent opens the PR.
  3. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Context pack update — Stage 3 implementation dispatched. Branch dzianisv/docs/agents-cua-mandate-239. PR pending; Stage 4 review follows once the PR is open and read back.

  4. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Context pack update — Stage 3 done. PR #240 open (head 6065a86, base cad292a), diff read back by the lead: only AGENTS.md, +1/-20, CI subsection deleted, MANDATORY block replaced with the REMOVED note verbatim. Stage 4 review dispatched; merge waits on the verdict + CI green.

  5. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Context pack update — Stage 4 PASS, verdict comment 6857890431 read back by the lead (head 6065a86, test-count delta 0/0/0). One record note: the diff is a single hunk, not two, because the two removed blocks are adjacent — scope is exactly the two blocks, nothing else. CI pending at check time: build in progress, iOS simulator build queued; test / typecheck / secret scan green. Merge waits on CI green.

  6. dzianisv commented on Nov 13, 2026

    @dzianisv
    OwnerAuthor

    Stage 5 merged: PR #240 merged as fb7344d (main HEAD now fb7344d), verified by readback. Stage 4 verdict PASS (comment 6857890431) on reviewed head 6065a86; CI green on 6065a86 (build, test, typecheck, secret scan, iOS simulator build success; build-fdroid/release skipped).

    Stage 5.5 — Design/PRD sync: checked — no update needed. This was a documentation-only change to AGENTS.md (removed a stale mandate); it alters no app behavior, contract, invariant, or PRD promise. No Design or PRD doc update required.

  7. added a commit that references this issue on Nov 13, 2026
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

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions