Resolve setup-in-workflow advisory on fix-lints Sweeps gem-install clause #124

New issue

Open

opened 2026-07-24 18:43:34 +00:00 by jared · 2 comments

jared commented

2026-07-24 18:43:34 +00:00

Owner

Surfaced during the lint-worklist checkpoint simplification (worktree-lint-sdlc-code). os-aidd-lint's setup-in-workflow cop flags the intentional 'gem install flay debride' recovery clause in plugins/os-sdlc/skills/fix-lints/SKILL.md's Sweeps section (advisory, non-blocking). The clause was deliberately moved out of a deleted Prerequisites section because its only non-obvious content is 'no Gemfile by design — never add one'. Decide: teach the cop to accept deliberately-placed recovery clauses outside Setup/Prerequisites (recognize more section names or an inline waiver), or accept the advisory as permanent noise. Cop: plugins/os-aidd-lint/lib/aidd_lint/cops/setup_in_workflow.rb

jared added the

labels

2026-08-01 16:39:16 +00:00

jared commented

2026-08-01 16:44:32 +00:00

Author

Owner

This was generated by AI during triage.

Verified against plugins/os-aidd-lint/lib/aidd_lint/cops/setup_in_workflow.rb: ALLOWED_HEADING still only matches prereq|setup|install|troubleshoot; there's no inline-waiver escape hatch. Not resolved.

Options: (A) widen ALLOWED_HEADING to also recognize a "Sweeps" (or generic "recovery"/"advisory") section name, (B) add an inline waiver comment/marker the cop respects anywhere, (C) accept as permanent noise.

Recommendation: (B). Widening the heading regex (A) papers over one instance but the cop's whole point is catching setup residue in arbitrary section names — the next deliberately-placed recovery clause will just need another heading added. An inline waiver keeps the heuristic general while giving authors a documented escape for the rare deliberate case.

> *This was generated by AI during triage.* Verified against `plugins/os-aidd-lint/lib/aidd_lint/cops/setup_in_workflow.rb`: `ALLOWED_HEADING` still only matches `prereq|setup|install|troubleshoot`; there's no inline-waiver escape hatch. Not resolved. Options: (A) widen `ALLOWED_HEADING` to also recognize a "Sweeps" (or generic "recovery"/"advisory") section name, (B) add an inline waiver comment/marker the cop respects anywhere, (C) accept as permanent noise. Recommendation: (B). Widening the heading regex (A) papers over one instance but the cop's whole point is catching setup residue in *arbitrary* section names — the next deliberately-placed recovery clause will just need another heading added. An inline waiver keeps the heuristic general while giving authors a documented escape for the rare deliberate case.

jared added the

plugin/os-aidd-lint

worklist/lint-rule

labels

2026-08-13 18:13:54 +00:00

jared added the

frozen

label

2026-08-16 14:31:00 +00:00

jared commented

2026-08-16 14:31:30 +00:00

Author