[PIX-7723] fix(done-gate): не считать git status кодовой доставкой в non-code SLA-check #60

Closed
andrei wants to merge 11 commits from fix/PIX-7723-done-gate-non-code-sla into master
Owner

Что сделано

Сужены regex-паттерны STRONG_CODE_TASK_PATTERN и EXPLICIT_CODE_DELIVERY_PATTERN в git-delivery-gate.ts: было \bgit\b (матчило любое слово "git"), стало git\s+push, git\s+merge, git\s+commit, git\s+diff и т.д. — только конкретные команды доставки.

Зачем

SLA-check агент поддержки указал в комментарии git status --short как runtime evidence ("изменений нет"). Старый паттерн ловил голое слово git, расценивал комментарий как code task и возвращал 422 для support-роли без push-прав.

План тестирования

  • Новый регрессионный тест does not treat git status --short runtime evidence in support SLA-check closeout as code delivery добавлен в issues-done-gate.test.ts
  • Остальные тесты gate не изменились

Где могу ошибаться

  • Паттерн branch убран из bare-слов. Если агент упомянет слово branch без git prefix — gate его не поймает. Но это edge case, который лучше чем ложные позитивы на git status.
## Что сделано Сужены regex-паттерны `STRONG_CODE_TASK_PATTERN` и `EXPLICIT_CODE_DELIVERY_PATTERN` в `git-delivery-gate.ts`: было `\bgit\b` (матчило любое слово "git"), стало `git\s+push`, `git\s+merge`, `git\s+commit`, `git\s+diff` и т.д. — только конкретные команды доставки. ## Зачем SLA-check агент поддержки указал в комментарии ``git status --short`` как runtime evidence ("изменений нет"). Старый паттерн ловил голое слово `git`, расценивал комментарий как code task и возвращал 422 для support-роли без push-прав. ## План тестирования - Новый регрессионный тест `does not treat git status --short runtime evidence in support SLA-check closeout as code delivery` добавлен в `issues-done-gate.test.ts` - Остальные тесты gate не изменились ## Где могу ошибаться - Паттерн `branch` убран из bare-слов. Если агент упомянет слово `branch` без `git ` prefix — gate его не поймает. Но это edge case, который лучше чем ложные позитивы на `git status`.
fix: ignore benign cancelled heartbeat runs in manager views
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m36s
8f7f9ad745
fix: redact inline secrets in agent command config
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m27s
59f3a5df13
fix: default paperclip cost breakdowns to current month
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m44s
5b182698e4
[PIX-7666] fix codex recovery CLI fallbacks
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m45s
76298244f3
fix: restore costs by agent usage aggregates
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m35s
158537d731
fix: backfill costs by agent from heartbeat usage
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m39s
531a2244ee
fix: restore cost attribution for usage-only agent spend
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m12s
9d7b0840b4
fix: normalize budget guardrail telemetry
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m33s
30f873ad82
fix: restore support ticket snapshot helper
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m33s
8b5da53d82
fix: ignore git status evidence in non-code done gate
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m20s
f4d72925b0
andrei closed this pull request 2026-06-11 07:26:46 +00:00
All checks were successful
Agents CI / Typecheck and Build (pull_request) Successful in 2m20s

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
europa-tech-srl/europa-tech-agents!60
No description provided.