Skip to content

feat: Wire source_issue param through create/update/batch surfaces#100

Merged
mabry1985 merged 1 commit into
mainfrom
feat/bd-87i
Jul 23, 2026
Merged

feat: Wire source_issue param through create/update/batch surfaces#100
mabry1985 merged 1 commit into
mainfrom
feat/bd-87i

Conversation

@mabry1985

@mabry1985 mabry1985 commented Jul 23, 2026

Copy link
Copy Markdown
Member

Summary

Wire source_issue param through create/update/batch surfaces (bd-87i)

See the diff for details.

Fixes #97

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA panel review — PASS

code-review-structural · head b0d125f783a7 · formal

[review-synthesizer completed: workflow code-review-structural:report]

Prose Brief

Overall risk: Low. A ~700-line purely additive change introducing new functionality with safe defaults and strict normalization gated before any mutation. All four LLM finders returned empty arrays, and the structural verifier spot-checked loop.py for the _source_issue import dependency and traced every code path — confirming zero defects.

Fix-first item: None. No findings survive verification; nothing needs fixing.

Panel disagreement: None. Unanimous empty-array verdicts across all finders and the structural pass.

What verification changed: Nothing. The verifier confirmed the clean slate — no findings were refuted, marked uncertain, or re-graded.

Gaps / Coverage note: No finder reported Gap: lines. The structural pass (code-review-structural:verify) completed successfully. With ~21 tests covering happy paths, rejection cases, batch isolation, and round-trip compatibility, coverage is thorough for an additive change of this kind.

[]

@protoreview protoreview Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Promoting the PASS verdict for head b0d125f783a7: all checks terminal-green, zero unresolved review threads. (approve-on-green)

@mabry1985
mabry1985 merged commit 0a4e972 into main Jul 23, 2026
1 check passed
@mabry1985
mabry1985 deleted the feat/bd-87i branch July 23, 2026 09:20
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.

source_issue is read as the explicit source-of-truth but unsettable through the board API — wire the param through create/update/batch

1 participant