Skip to content

refactor(ticket-192): extract helpers from merge_missing_ticket_records (STARTER-611) - #376

Merged
ifuri-validator-agent[bot] merged 1 commit into
mainfrom
ticket/192-planfile-compat-complexity
Sep 20, 2026
Merged

ifuri-validator-agent[bot] merged 1 commit into
mainfrom
ticket/192-planfile-compat-complexity

Conversation

@tom-sapletta-com

Copy link
Copy Markdown
Contributor

Reduce cyclomatic complexity of merge_missing_ticket_records from CC=25 (limit 15) to CC=3 by extracting behavior-preserving module-level helpers (_ticket_identifier, _normalized_status, _reported_ticket_ids, _already_covered, _decorated_recovery, _recover_missing_records, _compatibility_report).

  • Pure refactor: old-vs-new differential over 2420 generated scenarios (sprint layouts × status matrix × reported payload shapes) is byte-identical for the merged list and report.
  • Adds tests/test_planfile_compat.py (13 behavior-pinning tests); consumer suites (test_queue_clean, test_context, test_planfile_queue) pass unmodified.
  • ruff check clean; project/governance-check.sh --base origin/main = GOV-PASS.
  • Planfile source ticket: STARTER-611.

@ifuri-validator-agent ifuri-validator-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Validator approval after policy checks for exact head 641b49eceacb47c08dd53ab27e673579514e2112.

Ticket: ticket-192
Correlation ID: local-semcod-koru-pr-376-ticket-192
Model: zai/glm-5.3
Reviewed diff chunks: 2
Advisory LLM verdict: APPROVE
Advisory summary: Reviewed all 2 diff chunk(s). Chunk 1/2 of a behavior-preserving complexity refactor of merge_missing_ticket_records. Extracted helpers (_ticket_identifier, _normalized_status, _reported_ticket_ids, _already_covered, _decorated_recovery, _recover_missing_records, _compatibility_report) are private module-level functions with no public interface changes, no runtime dependencies added, and logic that matches the described original semantics (id/ticket_id fallback, strip, dedupe via seen_raw, reported-payload authority, legacy/unknown decoration, sprint marker removal, sorted status counts). Ticket intent metadata is complete and consistent with the diff. Required checks (onedev/local-verify, standard packs / conformance) all PASS per the protected assessment. | Refactor extracts helpers from merge_missing_ticket_records with behavior preserved (per PR differential testing), adds 13 thorough behavior-pinning tests. Required checks (onedev/local-verify, standard packs / conformance) pass. No security or blocking concerns visible in this chunk.
Advisory findings: none
The LLM output above is advisory and was not used as the approval trust root.
Semantic review prerequisite: not_required; policy 676cb4516bbfed2a000e40b9b1b6e4a430ecc761ec546aeb53d721a1905cfdd7.

Actual PR impact radar

Exact range: 17270fafdea8003c07d0dfd2a44fcb4b1867e6a4...641b49eceacb47c08dd53ab27e673579514e2112
Change digest: 2d141ff552d5567c7931424005ca0687fc3040fe653f07d4e68ad02021db42cb
Score: 56/100 (L), estimated 67 min, split recommended: true
Affected services/components: repository-wide/unclassified

Machine-readable radar JSONL and SVG
{"actual_change":{"additions":556,"base_sha":"17270fafdea8003c07d0dfd2a44fcb4b1867e6a4","binary_files":0,"categories":{"code":1,"configuration":1,"docs":2,"tests":1},"change_digest":"2d141ff552d5567c7931424005ca0687fc3040fe653f07d4e68ad02021db42cb","comparison":"17270fafdea8003c07d0dfd2a44fcb4b1867e6a4...641b49eceacb47c08dd53ab27e673579514e2112","deletions":46,"file_count":5,"files":["project/TICKETS.md","project/ticket-192/README.md","project/ticket-192/intent.json","src/koru/planfile_compat.py","tests/test_planfile_compat.py"],"head_sha":"641b49eceacb47c08dd53ab27e673579514e2112","service_count":0,"services":[]},"assessment_mode":"observed-pr","axes":{"coupling":4,"delivery":2,"scope":4,"uncertainty":3,"validation":1},"complexity":"L","confidence":0.9,"diagnostics":["RADAR-ACCEPTANCE-MISSING","RADAR-BUDGET-EXCEEDED"],"estimate":{"budget_minutes":30,"minutes":67,"within_budget":false},"impact":{"components":["origin","project","src/koru","tests"],"files":["origin/main","project/TICKETS.md","project/governance-check.sh","project/ticket-192/README.md","project/ticket-192/intent.json","src/koru/planfile_compat.py","tests/test_planfile_compat.py"],"public_interfaces":[],"runtime_dependencies":0},"schema":"subactor.ticket-radar/v1","score":56,"split":{"parts":[{"estimated_minutes":15,"name":"Implement origin","scope":["origin"]},{"estimated_minutes":15,"name":"Implement project","scope":["project"]},{"estimated_minutes":15,"name":"Implement src/koru","scope":["src/koru"]},{"estimated_minutes":15,"name":"Implement tests","scope":["tests"]},{"estimated_minutes":15,"name":"Validate and project to trackers","scope":["tests","planfile","github/gitlab/jira projections"]}],"reason":"estimated_minutes_exceed_budget","recommended":true},"standards":[{"id":"wellmanifest/dsl","revision":"6c60fc4e0dd1f1bb74f46a7745e28019908d1203","version":"0.1.0-dev"},{"id":"wellmanifest/ticket-lifecycle","revision":"5bf581907a87b46a13a73e6c033d3abe4d9a306f","version":"0.1.0-dev"},{"id":"wellmanifest/git-lifecycle","revision":"7d77d4b7af57e69bc75c3a0290b3a4805c5c4438","version":"0.2.0-dev"},{"id":"wellmanifest/logs","revision":"48c284ef7a069055c0bcb6b900147ce5e65f8b43","version":"0.3.0"}],"ticket_ref":"ticket-192"}
<svg xmlns="http://www.w3.org/2000/svg" width="128" height="128" viewBox="0 0 128 128" role="img"><title>ticket-192: refactor(ticket-192): extract helpers from merge_missing_ticket_records (STARTER-611)</title><rect width="128" height="128" rx="12" fill="#f8fafc"/><g stroke-width="1"><polygon points="64,55 72,61 69,71 59,71 56,61" fill="none" stroke="#d7dde5"/><polygon points="64,47 80,59 74,78 54,78 48,59" fill="none" stroke="#d7dde5"/><polygon points="64,38 89,56 79,85 49,85 39,56" fill="none" stroke="#d7dde5"/><polygon points="64,30 97,53 84,92 44,92 31,53" fill="none" stroke="#d7dde5"/><polygon points="64,21 105,51 89,99 39,99 23,51" fill="none" stroke="#d7dde5"/><line x1="64" y1="64" x2="64" y2="21" stroke="#aab4c0"/><line x1="64" y1="64" x2="105" y2="51" stroke="#aab4c0"/><line x1="64" y1="64" x2="89" y2="99" stroke="#aab4c0"/><line x1="64" y1="64" x2="39" y2="99" stroke="#aab4c0"/><line x1="64" y1="64" x2="23" y2="51" stroke="#aab4c0"/></g><polygon points="64,30 97,53 79,85 59,71 48,59" fill="#fb923c" fill-opacity="0.45" stroke="#c2410c" stroke-width="2"/><circle cx="64" cy="64" r="3" fill="#c2410c"/><g font-family="sans-serif" font-size="7" fill="#334155"><text x="64" y="11" text-anchor="middle">SCO</text><text x="114" y="48" text-anchor="middle">COU</text><text x="95" y="107" text-anchor="middle">UNC</text><text x="33" y="107" text-anchor="middle">VAL</text><text x="14" y="48" text-anchor="middle">DEL</text></g><text x="64" y="124" text-anchor="middle" font-family="sans-serif" font-size="8" fill="#0f172a">L · 67m</text></svg>
Merge will be attempted after this approval when explicitly authorized. ## Decision record (recomputable)
DECISION D-192-1038
TICKET ticket-192
HEAD_SHA 641b49eceacb47c08dd53ab27e673579514e2112
CORRELATION_ID local-semcod-koru-pr-376-ticket-192
ACTOR agent:ifuri-validator-agent[bot]
APPLIED_RULE P-CORE-015
INPUT author_login = "tom-sapletta-com"
INPUT observed_checks = ["governance / remote lifecycle=PASS","standard packs / conformance=PASS","governance / enforce=PASS","onedev/local-verify=PASS"]
INPUT required_checks = ["onedev/local-verify","standard packs / conformance"]
INPUT required_checks_source = "protected registry + GitHub applied rules (env/request)"
INPUT reviewer_login = "ifuri-validator-agent[bot]"
INPUT semantic_review_assessment = {"schema":"subactor.validator/semantic-review-assessment/v1","subject":{"repository":"semcod/koru","pull_request":376,"head_sha":"641b49eceacb47c08dd53ab27e673579514e2112","base_sha":"17270fafdea8003c07d0dfd2a44fcb4b1867e6a4","diff_sha256":"bf557a4fe294d9e7b00a47cad182f690afd71681635b686c084798b25afae447"},"policy":{"policy_schema":"subactor.validator/semantic-review-policy/v1","policy_version":1,"policy_sha256":"676cb4516bbfed2a000e40b9b1b6e4a430ecc761ec546aeb53d721a1905cfdd7","required":false,"critical_paths":[],"observed_paths":["project/TICKETS.md","project/ticket-192/README.md","project/ticket-192/intent.json","src/koru/planfile_compat.py","tests/test_planfile_compat.py"]},"grounding":"full-diff-not-per-finding-proof","execution_authority":false,"status":"not_required","reason":null,"review_sha256":null,"unresolved":[]}
INPUT superseded_checks = ["smoke"]
INPUT ticket_radar_receipt = {"schema":"subactor.ticket-radar/v1","base_sha":"17270fafdea8003c07d0dfd2a44fcb4b1867e6a4","head_sha":"641b49eceacb47c08dd53ab27e673579514e2112","change_digest":"2d141ff552d5567c7931424005ca0687fc3040fe653f07d4e68ad02021db42cb","score":56,"complexity":"L","estimated_minutes":67,"split_recommended":true,"services":[],"authority":"ADVISORY","promotion":"FORBIDDEN"}
VERDICT APPROVE AUTHORITY DETERMINISTIC
REJECTED REQUEST_CHANGES BECAUSE NO_UNSAFE_CHANGE_REASON_FOUND
ADVISORY llm_verdict = "APPROVE" MODEL "zai/glm-5.3"
ASSERT VERDICT_AUTHORITY != "ADVISORY"

@ifuri-validator-agent
ifuri-validator-agent Bot merged commit 9bbd7ff into main Sep 20, 2026
5 checks passed
@ifuri-validator-agent
ifuri-validator-agent Bot deleted the ticket/192-planfile-compat-complexity branch September 20, 2026 08:21
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.

1 participant