[Fleet Monitor] combsmith/google-app-scripts — .github/workflows/ci.yml #641

Activity

1. don-petry

   added

   health-check

   dev-leadFor dev-lead agent pickupFor dev-lead agent pickup

   fleet-tracker

   on Oct 21, 2026
2. don-petry commented on Oct 21, 2026

   

   don-petry

   on Oct 21, 2026

   CollaboratorAuthor

   More actions

   Dev-Lead Implementation Plan

   Issue: #641 — [Fleet Monitor] combsmith/google-app-scripts — .github/workflows/ci.yml

   Root cause

   The single failing run (39531241149, PR #637 standards-sync) failed in the autofix job: Prettier rewrote the newly-synced standards-owned stub .github/workflows/pr-review.yml (double → single quotes, collapsed the two-space gap before # NOSONAR), and the push was rejected:
   refusing to allow a GitHub App to create or update workflow .github/workflows/pr-review.yml without workflows permission.
   Even if the push had succeeded, it would have re-introduced byte drift against the canonical stub.

   Scope

   Exclude the standards-owned pr-review.yml caller stub from Prettier, mirroring the existing entries for dev-lead.yml, persona-mention.yml, agent-shield.yml, etc. (same approach as #618/#619).

   Implementation Checklist

   • .prettierignore: add .github/workflows/pr-review.yml with a justification comment

   Tests to Write

   • None — config-only change (TDD exception, same as feat: implement issue #618 — Add persona-mention.yml to .prettierignore before the stub sync #619). Verified by running npx prettier --check . against the PR chore: sync 2 org-standard workflow stub(s) from combsmith/.github #637 version of pr-review.yml.

   Test Command

   npm run check && npm test

   Notes

   PR #627 also adds this entry plus a broader autofix token rework; this issue only takes the minimal .prettierignore fix.
3. don-petry commented on Oct 21, 2026

   

   don-petry

   on Oct 21, 2026

   CollaboratorAuthor

   More actions

   Dev-Lead: Implementation Complete — PR #642

   The implementation for issue #641 is durable: commits are pushed and a pull request is open. This record is backed by verifiable artifacts you can confirm in one click.

   • Pull request: feat: implement issue #641 — [Fleet Monitor] combsmith/google-app-scripts — .github/workflows/ci.yml #642
   • Head commit: fe4940b70bdd5dfac349935256633afbf8f12f5c
   • Net change against main: 1 file(s), +7/-0 lines
   • Run: https://github.com/combsmith/google-app-scripts/actions/runs/39598615331

   Review happens on the PR. Acceptance criteria and test results are evidenced by the PR's diff and CI checks — not asserted here. If a later stage fails, this claim is retracted in place.
4. don-petry

   closed this as completedin #642on Oct 22, 2026
5. don-petry

   added a commit that references this issue on Oct 22, 2026

   feat: implement issue #641 — [Fleet Monitor] combsmith/google-app-scr…

   ...

   c62a348