Skip to content

PLF-10819: reconcile remote branch refactor/linker-split-006 - #110

Merged
ifuri-validator-agent[bot] merged 3 commits into
mainfrom
refactor/linker-split-006
Sep 1, 2026
Merged

PLF-10819: reconcile remote branch refactor/linker-split-006#110
ifuri-validator-agent[bot] merged 3 commits into
mainfrom
refactor/linker-split-006

Conversation

@tom-sapletta-com

Copy link
Copy Markdown
Contributor

Ticket: PLF-10819

Correlation ID: PLF-10819-autogrammar-todo2code-refactor-linker-split-006-3a7f52ea062b

Frozen HEAD: 3a7f52e
Branch: refactor/linker-split-006
Repository: autogrammar/todo2code
Authority source: governed_repair_ticket

Opened by bounded remote-branch discovery for an origin branch ahead of main with no open or closed pull request. Discovery does not merge, delete the branch, or infer GitHub authority from the branch name. Validator remains the publication boundary.

tom-sapletta-com and others added 2 commits September 1, 2026 13:09
Extract keyword indexing, candidate pairing, scoring, and relation rules
from linker.ts (STARTER-006) while keeping linkIntentRecords as the facade.

Co-authored-by: Cursor <cursoragent@cursor.com>
Reintroduce isOverlappingSameSourceProjection lost during linker module
split so document/nl excerpts from the same location link as documents.
ifuri-validator-agent[bot]
ifuri-validator-agent Bot previously approved these changes Sep 1, 2026

@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.

Deterministic Validator approval for exact head e7518a3d0d61f09ec5724e7dc02449cdf4f09f67.

Ticket: ticket-006
Correlation ID: todo2code-pr-110-ticket-006
Model: zai/glm-5.3
Reviewed diff chunks: 4
Advisory LLM verdict: APPROVE
Advisory summary: Reviewed all 4 diff chunk(s). Chunk 1 of 4 introduces new linker modules (candidate bucketing, keyword indexing, relation orientation, scoring helpers). Code is straightforward TypeScript with deduplicated pair generation, bounded bucket sizes, and documented heuristic logic. No security issues, no failing checks in test data. Remaining chunks reviewed separately. | This chunk moves linker scoring, keyword indexing, and candidate-pair collection out of linker.ts into focused modules (linker-scoring.ts, linker-keywords.ts, linker-candidates.ts). The visible scoring logic preserves original behavior: shared tickets (+0.62), shared symbols (+0.48, with resolved NL/AST symbol preference), path gating (+0.28 and module coverage only when paths intersect), same-action, text similarity, module-topic bonus, and same-source penalty. Deletion of the old inline implementations from linker.ts is consistent with the new imports; all listed tests pass. | Chunk 3 of the linker-split refactor removes scoring, bucketing, basename-resolution, and relation logic from linker.ts, presumably relocated into the new linker-candidates/linker-keywords/linker-relations/linker-scoring modules introduced in other chunks. The removed code carries well-documented, precision-oriented heuristics (unique-basename resolution, module-topic floor of 3 shared topics, same-source overlap suppression). No behavioral changes are inferable from this chunk alone. | Chunk 4 of 4 shows a small, benign helper function (countBy) with no security, correctness, or policy concerns visible in this segment. All listed checks pass.
Advisory findings: none
The LLM output above is advisory and was not used as the approval trust root.
Merge will be attempted after this approval when explicitly authorized.

Decision record (recomputable)

DECISION D-006-1430
TICKET ticket-006
HEAD_SHA e7518a3d0d61f09ec5724e7dc02449cdf4f09f67
CORRELATION_ID todo2code-pr-110-ticket-006
ACTOR agent:ifuri-validator-agent[bot]
APPLIED_RULE P-CORE-015
INPUT author_login = "tom-sapletta-com"
INPUT observed_checks = ["governance=SKIPPING","Live OpenRouter contract (opt-in)=SKIPPING","Java adapter (JDK 17 required)=PASS","verify=PASS","koru / code-review=PASS","verify=PASS","Java adapter (JDK 17 required)=PASS"]
INPUT required_checks = ["verify","Java adapter (JDK 17 required)","koru / code-review"]
INPUT required_checks_source = "protected registry + GitHub applied rules (env/request)"
INPUT reviewer_login = "ifuri-validator-agent[bot]"
INPUT superseded_checks = []
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"

Extend ticket-088 intent so governance resolves the linker refactor
diff to the active IN_PROGRESS ticket.

Co-authored-by: Cursor <cursoragent@cursor.com>

@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.

Deterministic Validator approval for exact head dd5fa3a080c56119848ba50397c7dfe8a77b5e2a.

Ticket: ticket-088
Correlation ID: todo2code-pr-110-ticket-088
Model: zai/glm-5.3
Reviewed diff chunks: 4
Advisory LLM verdict: APPROVE
Advisory summary: Reviewed all 4 diff chunk(s). Chunk 1 of 4: introduces intent. budget expansion and three new linker modules (candidates, keywords, relations) as part of a linker split refactor. No failing checks in test_results_data. Code is a straightforward extraction with no obvious behavior changes, security issues, or secret access. | Refactor chunk extracting linker scoring logic (linker-scoring.ts) out of linker.ts, including shared-target evidence scoring, path intersection with resolvable basenames, and keyword-based similarity. Logic appears to be a faithful move/refactor; all visible checks pass. | This chunk shows removal of linker helper functions (bucket indexing, pair suppression, path/symbol scoring, relation determination) as part of a linker module split. Code is well-commented with clear rationale for suppression heuristics and precision-oriented scoring. No security issues, no failing tests in provided results. | Chunk 4 of a linker refactor shows removal of helper functions (isOverlappingSameSourceProjection, relationForSourceKinds, matchSourceRule, orientRelation, intersects, intersectsAliases) with countBy retained, consistent with splitting logic into the new linker-* modules. No test failures; all checks PASS.
Advisory findings: none
The LLM output above is advisory and was not used as the approval trust root.
Merge will be attempted after this approval when explicitly authorized.

Decision record (recomputable)

DECISION D-088-2490
TICKET ticket-088
HEAD_SHA dd5fa3a080c56119848ba50397c7dfe8a77b5e2a
CORRELATION_ID todo2code-pr-110-ticket-088
ACTOR agent:ifuri-validator-agent[bot]
APPLIED_RULE P-CORE-015
INPUT author_login = "tom-sapletta-com"
INPUT observed_checks = ["Live OpenRouter contract (opt-in)=SKIPPING","governance=SKIPPING","Java adapter (JDK 17 required)=PASS","verify=PASS","verify=PASS","Java adapter (JDK 17 required)=PASS","koru / code-review=PASS","verify=PASS","Java adapter (JDK 17 required)=PASS"]
INPUT required_checks = ["verify","Java adapter (JDK 17 required)","koru / code-review"]
INPUT required_checks_source = "protected registry + GitHub applied rules (env/request)"
INPUT reviewer_login = "ifuri-validator-agent[bot]"
INPUT superseded_checks = []
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 dcbf9ef into main Sep 1, 2026
17 checks passed
@ifuri-validator-agent
ifuri-validator-agent Bot deleted the refactor/linker-split-006 branch September 1, 2026 12:12
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