Created
Jul 04, 20:58
Started
Jul 04, 20:59
Completed
Jul 04, 22:40
DevOps handoff
Type
Feature
Shape
ui-only
Worktree Slug
completed-stage-ago-time
Repositories
mcritchie-studio
Release Slug
—
Branch
feat/completed-stage-ago-time
Acceptance Criteria
Expected Test Plan
Checks Run
Agent Context
Release pizza-tracker (app/views/tasks/_release_tracker.html.erb): active nodes tick a count-up via data-release-ticker/_release_ticker.html.erb; complete nodes render a STATIC start→complete duration (release_static_duration_label). Operator wants complete nodes to instead show time SINCE the stage finished (compact 'ago'), still live-ticking between LiveBoardFx card swaps, with the absolute completion timestamp as a title tooltip. Plan: release_tracker_duration :complete branch also returns completed_at; view renders data-release-ticker data-mode=ago data-since=completed_at; ticker JS gains an ago mode whose formatter MUST mirror the new server helper so first paint and first tick agree. Touch: application_helper.rb, _release_tracker.html.erb, _release_ticker.html.erb.
Stage Timeline
Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. — means the agent didn't report that metric.
Conversation
QA review feedback, agent handoffs, and follow-up notes for this task.
Scout report: merge-ready - Light review found no blockers: acceptance, diff, tests, CI, and merge safety all pass for PR #381. Reporter: alex Outcome: merge-ready Findings: - blocker: none - non-blocker: PR #380 also touches app/helpers/application_helper.rb and PR #378 touches test/helpers/application_helper_test.rb, but the hunks are disjoint; merge-tree checks are clean. - question: none Questions: - none Checks: - bin/rails test test/helpers/application_helper_test.rb — passed locally, 49 runs / 404 assertions - bin/dor-check completed-stage-ago-time — DoR-to-Merge met for ui-only - gh pr checks 381 --repo amcritchie/mcritchie-studio — scan_ruby, scan_js, lint, and test all passed - curl -fsS http://localhost:3001/up — returned 200 health page
Scout report: merge-ready - Primary UI review found PR 381 meets the completed-stage ago-time acceptance criteria and is clean to merge after green CI. Reporter: shannon Outcome: merge-ready Findings: - No blockers: completed tracker nodes render live ago labels from completion stamps, active node keeps count-up mode, and tooltip carries Finished timestamp plus took duration. - Merge safety: branch is a one-commit delta from release; open PR filename overlaps with #380/#378 merge cleanly and touch separate hunks. Questions: - none Checks: - bin/rails test test/helpers/application_helper_test.rb passed locally: 49 runs, 404 assertions. - bin/dor-check completed-stage-ago-time passed for ui-only DoR; full-suite and rubocop certifications present in checks_run. - GitHub PR checks pass: scan_ruby, scan_js, lint, test; mergeStateStatus CLEAN. - Local UI probe on http://localhost:3001/deployments found two complete ago tickers, one active count-up ticker, no console/page errors, and ago ticker advanced across a boundary.
pr-review moved completed-stage-ago-time to reviewed after shannon + alex reported merge-ready. Left for Steffon QA. Reports: - shannon: merge-ready - Primary UI review found PR 381 meets the completed-stage ago-time acceptance criteria and is clean to merge after green CI. - alex: merge-ready - Light review found no blockers: acceptance, diff, tests, CI, and merge safety all pass for PR #381.
Sealed-bid sizing
Edit →Alex (PM)
—
Avi (PO)
SMALL
Dev
SMALL
Actual
LARGE
We emailed a one-tap sign-in link to . It expires shortly and can only be used once.
No email? Check spam, or close this and try again.