PM stack: extend tests to the live readers, the comment consumer loop, and the deploy scripts #1213

Activity

1. dezren39

   added

   kind/testTest coverage, test infrastructure, or test-only fixesTest coverage, test infrastructure, or test-only fixes

   status/needs-triageNot yet triaged; needs a first lookNot yet triaged; needs a first look

   priority/normalOrdinary priority (the default)Ordinary priority (the default)

   app/pm-stackpm-stack (Pad/Grist) in developing-today/codepm-stack (Pad/Grist) in developing-today/code

   on Oct 27, 2026
2. developing-today commented on Oct 27, 2026

   

   developing-todayBot

   on Oct 27, 2026 – with developing.today

   Contributor

   More actions

   CODE-347
3. dezren39

   added a parent issue on Oct 27, 2026

   • epic(dashboard): PM stack coherence - single-URL, fresh data, triage-able, hierarchical #1184
4. github-actions

   assigned

   dezren39

   on Oct 27, 2026
5. dezren39

   added

   source/humanFiled directly by a humanFiled directly by a human

   effort/lLarge: multi-day or open-ended; expect staged check-insLarge: multi-day or open-ended; expect staged check-ins

   meta/enrichedIntake review done: acceptance criteria, links, and normalized labels addedIntake review done: acceptance criteria, links, and normalized labels added

   on Oct 28, 2026
6. dezren39 commented on Oct 28, 2026

   

   dezren39

   on Oct 28, 2026

   MemberAuthor

   More actions

   Intake review

   Fleet-monitor intake pass (protocol §5), 2026-10-28.

   Still live: verified this run — the issue is open, and a merged-PR search found no PR that explicitly resolves it (no fixes-reference; the only real merged hit in the search set, PR #1318, does not mention it). It does not merely look solved.

   Related: Related: epic #1184; extends the existing stub-server pattern used for projector/opencode.

   Notes / risks: Risk: the end-to-end comment-loop test depends on a consumer that does not exist yet — land the live-reader and script tests first; the loop test is explicitly conditional.

   Suggested approach: Suggested approach: live-reader stubs first, seed.sh idempotence second, combined CI gate third.

   Labels normalized this pass (see the issue's label set); acceptance criteria appended to the body as the standard ## Acceptance criteria section, original body preserved verbatim above it.
7. dezren39

   added

   work/assignedQueue worker lifecycle stateQueue worker lifecycle state

   queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

   status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

   and removed

   status/needs-triageNot yet triaged; needs a first lookNot yet triaged; needs a first look

   on Nov 1, 2026
8. 112 remaining items

   Load more
9. dezren39

   added

   queue/parkedQueue task is parked (waiting on CI, time, or an event)Queue task is parked (waiting on CI, time, or an event)

   status/parkedDeliberately paused with a recorded resume conditionDeliberately paused with a recorded resume condition

   on Nov 2, 2026
10. dezren39 commented on Nov 2, 2026

    

    dezren39

    on Nov 2, 2026

    MemberAuthor

    More actions

    🤖 Agent note · parked

    • Agent: queue worker (repo-worker-code), lane B shift
    • Session / ID: task 4935
    • Found via: queue claim
    • Task: Parked task 4935 for 2h — Drew-owned canonical PR hard stop rechecked fresh at 2026-11-02T06:34Z: issue PM stack: extend tests to the live readers, the comment consumer loop, and the deploy scripts #1213 is OPEN and assigned to dezren39; PR test(pm-stack): cover remaining #1213 gaps #4011 is OPEN, authored by dezren39, head 2296432. No competing work was created and PR test(pm-stack): cover remaining #1213 gaps #4011, its branch, and any worktree were not inspected or modified.
    • Expectation: Resume only after Drew releases or hands off PR test(pm-stack): cover remaining #1213 gaps #4011; revalidate ownership, PR state, and checks then.
    • Links: PR test(pm-stack): cover remaining #1213 gaps #4011
11. dezren39

    added

    queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

    status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

    and removed

    status/parkedDeliberately paused with a recorded resume conditionDeliberately paused with a recorded resume condition

    queue/parkedQueue task is parked (waiting on CI, time, or an event)Queue task is parked (waiting on CI, time, or an event)

    on Nov 2, 2026
12. dezren39 commented on Nov 2, 2026

    

    dezren39

    on Nov 2, 2026

    MemberAuthor

    More actions

    🤖 Agent note · picked up

    • Agent: queue worker (repo-worker-code), lane E shift
    • Session / ID: task 4935
    • Found via: queue claim (resume from checkpoint: PR test(pm-stack): cover remaining #1213 gaps #4011 opened)
    • Task: Resuming task 4935's own PR test(pm-stack): cover remaining #1213 gaps #4011 (branch queue/4935-1213-test-gaps). Correction to the previous park: PR test(pm-stack): cover remaining #1213 gaps #4011 is this task's fleet PR — branch name and task binding say so; the dezren39 author login is the shared account and is not Drew-ownership (the standing false-positive pattern). Remaining work: disposition the two red checks on head 229643221 — the new gate's ready-for-agent smoke checks (this PR's harness) and the aggregator Typecheck step (files this PR does not touch; verifying against main) — then merge when verified.
    • Expectation: effort/l; done = PR test(pm-stack): cover remaining #1213 gaps #4011 merged and issue PM stack: extend tests to the live readers, the comment consumer loop, and the deploy scripts #1213 closed, or parked with the exact failing step named.
    • Links: issue PM stack: extend tests to the live readers, the comment consumer loop, and the deploy scripts #1213 · PR test(pm-stack): cover remaining #1213 gaps #4011 · task 4935
13. dezren39

    added 2 commits that reference this issue on Nov 2, 2026

    test(pm-stack): model systemctl start timestamps in install smoke (#1213

    ...

    2f9ef0e

    Merge pull request #4011 from developing-today/queue/4935-1213-test-gaps

    ...

    cc819fe
14. dezren39 commented on Nov 2, 2026

    

    dezren39

    on Nov 2, 2026

    MemberAuthor

    More actions

    🤖 Agent note · done

    • Agent: queue worker (repo-worker-code), lane E shift
    • Session / ID: task 4935
    • Found via: queue claim (resume from checkpoint: PR test(pm-stack): cover remaining #1213 gaps #4011 opened)
    • Task: All three remaining gaps of issue PM stack: extend tests to the live readers, the comment consumer loop, and the deploy scripts #1213 are covered and merged via PR test(pm-stack): cover remaining #1213 gaps #4011 (merge commit cc819feb8):
      1. Stubbed-fetch tests for the live Grist roadmap reader (test/grist-live.test.ts: mapping + bearer auth + 503 fixture fallback/provenance).
      2. Seed/deploy script coverage: seed-smoke.test.sh (Grist + Pad seed idempotency — second run performs no creates) and deploy-install-smoke.test.sh (aggregator + ready-for-agent install/render smoke under temporary HOME with stubbed commands).
      3. Combined CI gate .github/workflows/pm-stack-tests.yml running the aggregator + projector Bun suites and both shell harnesses together.
    • Verification: the new gate's run on the final head 2f9ef0ed4 is SUCCESS (run 41424221066); on the laptop both shell harnesses pass all checks and bun test test/grist-live.test.ts test/slice3.test.ts is 28 pass / 0 fail. The Codex P2 on the workflow was fixed in 229643221 and replied on its thread. One fix this shift: main's c515c9d33 changed ready-for-agent/install.sh (restart proven via ExecMainStartTimestampMonotonic), so the harness systemctl stub now models the start timestamp and the assertion matches the new success line.
    • Not this PR: the aggregator Typecheck step is red from the pre-existing main baseline — exactly 12 tsc errors, all in main-side files and none in files this PR touches — already tracked in issue aggregator: tsc --noEmit red on main — 12-error baseline (11 in tests, web-push types missing) #4023.
    • Links: PR test(pm-stack): cover remaining #1213 gaps #4011 · task 4935 · issue aggregator: tsc --noEmit red on main — 12-error baseline (11 in tests, web-push types missing) #4023
15. dezren39

    closed this as completedon Nov 2, 2026
16. dezren39

    removed

    status/in-progressQueue worker lifecycle stateQueue worker lifecycle state

    queue/claimedQueue worker lifecycle stateQueue worker lifecycle state

    work/assignedQueue worker lifecycle stateQueue worker lifecycle state

    on Nov 2, 2026