Repository navigation
tech-debt(ci): workflow trigger paths drift — auto-detect missing path globs (#7137 follow-up) #7223
Description
Context
#7137 (PR #7138) needed a second commit to add `autobot-slm-backend/ansible/ansible.cfg` to the `ansible-role-facts-test.yml` workflow's trigger paths. Without it, the fix didn't trigger CI verification — silent green via "workflow not triggered" instead of green via "tests pass".
Problem
There's no system-wide check that workflow trigger paths cover all files those workflows actually validate. Changes to ancillary files (configs, helpers) can bypass the workflow that's supposed to verify them.
Proposed solution
CI lint step that:
- For each workflow, parses `paths:` glob list
- Checks that all imports/refs in the workflow's run: scripts are covered by at least one glob
- Flags workflows where touching the actual code-under-test wouldn't trigger the workflow
OR simpler heuristic: a workflow that runs `ansible-playbook` with `-i autobot-slm-backend/ansible/inventory/...` SHOULD have `autobot-slm-backend/ansible/**` in its paths.
Acceptance criteria
- Lint identifies workflows where trigger paths are narrower than the files they verify
- Existing workflows audited (likely 2-3 with the same pattern)
- Per-workflow comment documenting the rationale for the path subset
Activity
ci: extend ansible-role-facts-test trigger paths (closes #7223 partial)
ci: extend ansible-role-facts-test trigger paths (closes #7223 partia…
github-actions commentedon May 8, 2026
PR #7242 (merged to
Dev_new_gui) references this issue with a close keyword.If this issue is fully resolved, close it manually. If work remains, no action is needed.
mrveiss commentedon May 8, 2026
✅ Closed with proof — commit c03d01834 (PR #7242).
ACs Met:
Note: Auto-detect lint (acceptance criterion 1) was deferred — would require parsing each workflow's run: scripts to identify which files they touch, which is heavy. The fix-the-known-gaps approach handles the immediate impact; future regressions will surface when workflows fail to trigger on relevant changes.