PLF-10819: reconcile remote branch refactor/linker-split-006 - #110
Conversation
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.
There was a problem hiding this comment.
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>
There was a problem hiding this comment.
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"
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
mainwith 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.