Greptile Summary

This PR fixes a silent no-op bug in the gateway config hot-reload system: changes to agents.defaults.models (the model allowlist array) were never triggering a heartbeat restart because no explicit rule existed for that path, causing it to fall through to the agents catch-all no-op rule in BASE_RELOAD_RULES_TAIL.

The fix adds a dedicated agents.defaults.models reload rule with restart-heartbeat action, inserted before the existing agents.defaults.model (singular) rule. The rule ordering is safe — neither rule can shadow the other because the prefix-match condition requires either an exact match or a .-delimited child path, and "agents.defaults.models" does not start with "agents.defaults.model.".

Key changes:

• Added agents.defaults.models hot-reload rule in BASE_RELOAD_RULES (config-reload-plan.ts)
• Added a focused regression test confirming the path sets restartHeartbeat=true and is absent from noopPaths (config-reload.test.ts)
• All 22 existing tests continue to pass

Confidence Score: 5/5

• Safe to merge — minimal, targeted bug fix with no risk of regressions.
• This PR is a single rule insertion in an ordered list. The prefix-matching logic ensures no shadowing between the new agents.defaults.models rule and the existing agents.defaults.model rule. The new test directly validates the previously broken behavior, and all existing tests pass. Code changes are minimal and focused on fixing the identified issue.
• No files require special attention

_{Last reviewed commit: a615f0c}

Thanks again for this work.

This was landed indirectly via the synthesized PR #33831, and your contribution is credited in the changelog and as a co-author on the merge commit.

This PR merge/triage workflow is AI-assisted.

Closing this PR as superseded by #33831. If anything here looks incorrect or incomplete, reply to reopen and we can reassess.