Skip to content

agents: restore missing rule frontmatter and refine CI/rule guidance - #4387

Merged
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:learn_pr_4371_takeaways
Oct 31, 2026
Merged

rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:learn_pr_4371_takeaways

Conversation

@rickeylev

@rickeylev rickeylev commented Oct 31, 2026 •

Copy link
Copy Markdown
Collaborator

Several agent rules lacked YAML trigger frontmatter, preventing agent
discovery, while CI monitoring and merge workflows alerted on expected
merge-blocking label gates, woke parent conversations early during
failure analysis, and contended over monitor locks. Domain rules also
omitted guidance on executable target forwarding, Windows .exe CLI
output matching, and bidirectional alias cross-references.

Add model_decision frontmatter to unindexed rules and document those
Starlark, Windows, and documentation conventions. Update CI monitoring
and merge skills to ignore expected blocks-do-not-merge failures when
the merge-blocking label is active, scope failure analyzer alerts to
the analyzer subagent, and release active CI monitor locks before
launching the merge shepherd.

Several agent rules lacked YAML trigger frontmatter, preventing agent
discovery, while CI monitoring and merge skills alerted on expected
`do not merge` gates and contended over background monitor locks. In
addition, Starlark, Windows, and documentation rules omitted guidance
on forwarding executable targets, matching `.exe` suffixes in CLI
output, and cross-linking public aliases bidirectionally.

Add `model_decision` frontmatter to the unindexed rule files, document
executable forwarding, `.exe` output matching, and bidirectional alias
cross-references, skip expected `blocks-do-not-merge` failures when the
`do not merge` label is active, and hand off background CI monitors
before launching the merge shepherd.
@rickeylev

Copy link
Copy Markdown
Collaborator Author

AI Disclosure: This PR was prepared with AI assistance (semi-autonomous,
directed and reviewed by @rickeylev). Review comments on this PR will be
reviewed and addressed with human oversight before posting responses or pushing
changes.

This is an automatically generated agent comment.

@rickeylev
rickeylev enabled auto-merge October 31, 2026 04:17
@rickeylev
rickeylev merged commit ca713c1 into bazel-contrib:main Oct 31, 2026
14 of 15 checks passed
@rickeylev
rickeylev deleted the learn_pr_4371_takeaways branch October 31, 2026 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant