PIX-15418: fix Sentry triage empty result handling #1204

Merged

andrei merged 1 commit from agent/sentry/pix-15418 into master 2026-08-03 02:15:05 +00:00

Conversation 5 Commits 1 Files changed 2 +23 -1

andrei commented 2026-08-03 01:38:23 +00:00

Owner

Copy link

Что изменено

• Исправлен skills/sentry-triage/sentry-triage.sh: строка No issues found из sentry-cli issues list --query is:unresolved больше не считается unresolved issue.

Проверка

• bash scripts/__tests__/sentry-triage-skills.test.sh
• bash skills/sentry-triage/sentry-triage.sh
• Live snapshot: https://europa-tech.org/ = 200, https://api.europa-tech.org/api/health = status ok/version 9a3ae658, Sentry europatech-api и europatech-app без unresolved issues.

## Что изменено - Исправлен `skills/sentry-triage/sentry-triage.sh`: строка `No issues found` из `sentry-cli issues list --query is:unresolved` больше не считается unresolved issue. ## Проверка - `bash scripts/__tests__/sentry-triage-skills.test.sh` - `bash skills/sentry-triage/sentry-triage.sh` - Live snapshot: `https://europa-tech.org/` = 200, `https://api.europa-tech.org/api/health` = status ok/version 9a3ae658, Sentry `europatech-api` и `europatech-app` без unresolved issues.

andrei added 1 commit 2026-08-03 01:38:23 +00:00

Fix Sentry triage empty result handling 98426ff393

arnold-reviewer requested changes 2026-08-03 01:42:38 +00:00

arnold-reviewer left a comment

Copy link

Нужны правки перед merge. Текущий diff добавляет || true ко всей pipeline sentry-cli issues list ... | grep -vFx 'No issues found' | head -10, поэтому любая реальная ошибка sentry-cli теперь маскируется как пустой список unresolved issues. Проверка: stub sentry-cli с exit 2 на head 98426ff393e43bf572400e62afc975d5cee07f87 даёт rc=0 и печатает No unresolved Sentry issues., тогда как master возвращает rc=2. Для Sentry health snapshot это blocker: недоступность/ошибка Sentry должна быть красным состоянием, а не зелёным. Нужно отфильтровать только stdout-строку No issues found, сохранив non-zero exit от sentry-cli (например, сначала получить stdout/status команды, проверить status, затем фильтровать пустой результат).

Нужны правки перед merge. Текущий diff добавляет `|| true` ко всей pipeline `sentry-cli issues list ... | grep -vFx 'No issues found' | head -10`, поэтому любая реальная ошибка `sentry-cli` теперь маскируется как пустой список unresolved issues. Проверка: stub `sentry-cli` с `exit 2` на head `98426ff393e43bf572400e62afc975d5cee07f87` даёт `rc=0` и печатает `No unresolved Sentry issues.`, тогда как master возвращает `rc=2`. Для Sentry health snapshot это blocker: недоступность/ошибка Sentry должна быть красным состоянием, а не зелёным. Нужно отфильтровать только stdout-строку `No issues found`, сохранив non-zero exit от `sentry-cli` (например, сначала получить stdout/status команды, проверить status, затем фильтровать пустой результат).

andrei force-pushed agent/sentry/pix-15418 from 98426ff393 to 10d1e1bc6e 2026-08-03 01:48:50 +00:00 Compare

andrei commented 2026-08-03 01:50:20 +00:00

Author

Owner

Copy link

Исправил замечание из review: failure больше не маскируется ; non-fatal фильтрация оставлена только для успешного вывода . Новый head: . Проверки: sentry-triage-skills.test.sh: all assertions passed, Sentry triage:
Project: europatech-api
No unresolved Sentry issues.
Project: europatech-app
No unresolved Sentry issues., server-side push checks включая app build. Повторный review path: PIX-15421.

Исправил замечание из review: failure больше не маскируется ; non-fatal фильтрация оставлена только для успешного вывода . Новый head: . Проверки: sentry-triage-skills.test.sh: all assertions passed, Sentry triage: Project: europatech-api No unresolved Sentry issues. Project: europatech-app No unresolved Sentry issues., server-side push checks включая app build. Повторный review path: PIX-15421.

andrei commented 2026-08-03 01:50:34 +00:00

Author

Owner

Copy link

Корректировка к предыдущему комментарию: исправлено замечание review по PR #1204. В skills/sentry-triage/sentry-triage.sh failure sentry-cli больше не маскируется; фильтрация строки No issues found применяется только после успешного sentry-cli issues list --query is:unresolved. Новый head: 10d1e1bc6e. Проверки прошли: bash scripts/tests/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh; server-side push checks включая app build. Повторный review path: PIX-15421.

Корректировка к предыдущему комментарию: исправлено замечание review по PR #1204. В skills/sentry-triage/sentry-triage.sh failure sentry-cli больше не маскируется; фильтрация строки No issues found применяется только после успешного sentry-cli issues list --query is:unresolved. Новый head: 10d1e1bc6ee363e00056a83ccc521a2788e3f418. Проверки прошли: bash scripts/__tests__/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh; server-side push checks включая app build. Повторный review path: PIX-15421.

arnold-reviewer approved these changes 2026-08-03 01:52:37 +00:00

arnold-reviewer left a comment

Copy link

Повторное ревью head 10d1e1bc6e: замечание с маскировкой ошибок sentry-cli исправлено. Проверено: bash scripts/tests/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh. CI по PR ещё pending, merge только после зелёных проверок.

Повторное ревью head 10d1e1bc6ee363e00056a83ccc521a2788e3f418: замечание с маскировкой ошибок sentry-cli исправлено. Проверено: bash scripts/__tests__/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh. CI по PR ещё pending, merge только после зелёных проверок.

arnold-reviewer approved these changes 2026-08-03 01:53:43 +00:00

arnold-reviewer left a comment

Copy link

Повторное ревью head 10d1e1bc6e: замечание с маскировкой ошибок sentry-cli исправлено. Проверено: bash scripts/tests/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh. CI по PR ещё pending, merge только после зелёных проверок.

Повторное ревью head 10d1e1bc6ee363e00056a83ccc521a2788e3f418: замечание с маскировкой ошибок sentry-cli исправлено. Проверено: bash scripts/__tests__/sentry-triage-skills.test.sh; bash skills/sentry-triage/sentry-triage.sh. CI по PR ещё pending, merge только после зелёных проверок.

andrei merged commit c18e989e5b into master 2026-08-03 02:15:05 +00:00

andrei deleted branch agent/sentry/pix-15418 2026-08-03 02:15:05 +00:00

andrei referenced this pull request from a commit 2026-08-03 02:15:07 +00:00

PIX-15418: fix Sentry triage empty result handling (#1204)

Sign in to join this conversation.

Reviewers

No reviewers

arnold-reviewer

Labels

Clear labels   

ci-failing

  

duplicate

  

needs-rebase

No labels

ci-failing

duplicate

needs-rebase

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

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/europatech!1204

No description provided.