Skip to content

chore: release - merge dev into main - #1529

Merged
zbigniewsobiecki merged 2 commits into
mainfrom
dev
Aug 4, 2026
Merged

chore: release - merge dev into main#1529
zbigniewsobiecki merged 2 commits into
mainfrom
dev

Conversation

@zbigniewsobiecki

Copy link
Copy Markdown
Member

Automated release PR created by the release workflow.

Commits (2):

5d059228 fix(jira): match status by locale-invariant ID with name fallback (MNG-1768) (#1528)
eb8f36a5 fix(jira): paginate searchProjects() + make Combobox name-searchable (#1524)

aaight and others added 2 commits August 4, 2026 13:24
…1524)

* fix(jira): paginate searchProjects() + make Combobox name-searchable

* fix(ci): bump brace-expansion override to ^5.0.9 to clear high-severity audit

npm audit --omit=dev --audit-level=high flagged brace-expansion@5.0.8
(GHSA-rgw5-rvv9-x895, DoS via unbounded intermediate arrays). Bump the
existing override to ^5.0.9 so the transitive dependency (via
@oclif/core -> minimatch) resolves to the patched release.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Cascade Bot <bot@cascade.dev>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
…G-1768) (#1528)

* fix(jira): match status by locale-invariant ID with name fallback (MNG-1768)

* fix(jira): skip jira_transition_not_found capture on benign no-op moves

Address review feedback on PR #1528 (MNG-1768):

- moveWorkItem no longer fires the jira_transition_not_found Sentry
  capture when the issue is already in the destination status. Best-effort
  callers (createWorkItem's backlog move, lifecycle moveOnPrepare/
  moveOnSuccess) move unconditionally and JIRA offers no self-transition,
  so an already-there issue legitimately reaches the miss path — capturing
  there diluted the genuine locale/misconfig signal. Added isAlreadyInStatus
  helper (ID-first, name fallback) so only a real miss is surfaced loudly.
- Dropped the vestigial `t.id === destination` (transition-ID) matcher
  branch that could collide with a numeric status-ID destination; no caller
  passes a transition ID.
- Docs: src/integrations/README.md and docs/architecture/04-agent-system.md
  now reference resolvePMStatusAgentByIdOrNameFromWorkflowDefinitions for the
  JIRA status-changed trigger.
- Clarified the overstated pm-status "prefers the ID match" test comment
  (per-entry ID-before-name, first matching entry wins during iteration).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* fix(jira): match status by id-or-name in label-added and comment-mention (MNG-1768)

The MNG-1768 write side persists locale-invariant JIRA status IDs into
`jira.statuses`, but only status-changed dispatch and moveWorkItem were
migrated to match by ID. Two other readers of the same map still matched
the issue's `status.name` by name only, so they silently stopped firing
for ID-based configs (all new projects, plus any re-saved project):

- `JiraReadyToProcessLabelTrigger` (cascade-ready label flow)
- `JiraCommentMentionTrigger.isInPlanningStatus` (planning gate)

Add `resolvePMLabelAgentByStatusIdOrNameFromWorkflowDefinitions`
(mirroring the shared status resolver), read `status.id` alongside
`status.name` in both triggers, and gate planning by ID-first with a
case-insensitive name fallback. Now every consumer of the ID-valued
`jira.statuses` map is locale-invariant. Adds ID-config unit tests for
each trigger and the new resolver, and documents the two readers in the
MNG-1768 README subsection.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Cascade Bot <bot@cascade.dev>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
@zbigniewsobiecki
zbigniewsobiecki merged commit 494b144 into main Aug 4, 2026
10 of 11 checks passed
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.60000% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pm/jira/adapter.ts 78.12% 7 Missing ⚠️
src/triggers/jira/status-changed.ts 94.11% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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.

2 participants