mathias/hyperguild
1
0
Fork 0
Code Issues 30 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

CanonicalizeLinks silently leaves wing:/wiki/-prefixed wikilinks broken instead of repairing them #87

New Issue
Closed
opened 2026-07-26 21:12:24 +00:00 by mathias · 1 comment
mathias commented 2026-07-26 21:12:24 +00:00
Owner
Copy Link
Copy Source

Problem

4 wikilinks have a literal path-root prefix baked into the target text instead of a clean slug:

  • wiki/homelab/failures/act-runner-host-mode-container-needs-node-and-libatomic.md → [[wing:homelab/failures/rootless-buildah-act-runner-run-containers-denied]]
  • same file → [[wing:homelab/failures/act-runner-host-executor-tmp-persists]]
  • wiki/agentsquad/hypotheses/council-consolidation-standalone-deliberation-service.md → [[wiki/agentsquad/decisions/autoresearch-council-sibling-pipe]]
  • same file → [[wiki/agentsquad/facts/llm-council-design-and-first-runs-2026-06-21]]

Both targets resolve correctly once the wing:/wiki/ prefix is stripped — the underlying notes exist.

Where it lives

ingestion/internal/pipeline/links.go, CanonicalizeLinks/canonicalizeContent. plainLinkRE matches any [[...]] without a pipe, including these path-prefixed forms, and treats the whole string as a "display name" to look up in titleToSlug (built from note titles, not paths). Lookup fails, it emits an "unknown wikilink" warning and leaves the malformed link untouched — so however the LLM extraction step originally produced a path-style link (likely a schema/prompt slip, not this function), this canonicalization pass has no path to actually repair it.

Scope

Before falling back to the title-lookup path, detect and strip a leading wing: or wiki/ (or any known root prefix) from the bracket content, and try resolving the remainder as a path-style wing/hall/slug or hall/slug target directly (same resolution style the audit tooling in brain-gardener already uses per knowledge/brain-graph-key-by-path-not-basename-slug.md). Also worth checking BuildPrompt's schema instructions — if the LLM is told exact link syntax and still produces prefixed forms, the schema/prompt may need tightening too.

Scope (repair)

Fix the 4 existing links to clean hall/slug form once the normalizer handles this case.

Acceptance criteria

  • Regression test covering both prefix shapes resolving correctly.
  • The 4 links repaired.
## Problem 4 wikilinks have a literal path-root prefix baked into the target text instead of a clean slug: - `wiki/homelab/failures/act-runner-host-mode-container-needs-node-and-libatomic.md` → `[[wing:homelab/failures/rootless-buildah-act-runner-run-containers-denied]]` - same file → `[[wing:homelab/failures/act-runner-host-executor-tmp-persists]]` - `wiki/agentsquad/hypotheses/council-consolidation-standalone-deliberation-service.md` → `[[wiki/agentsquad/decisions/autoresearch-council-sibling-pipe]]` - same file → `[[wiki/agentsquad/facts/llm-council-design-and-first-runs-2026-06-21]]` Both targets resolve correctly once the `wing:`/`wiki/` prefix is stripped — the underlying notes exist. ## Where it lives `ingestion/internal/pipeline/links.go`, `CanonicalizeLinks`/`canonicalizeContent`. `plainLinkRE` matches any `[[...]]` without a pipe, including these path-prefixed forms, and treats the whole string as a "display name" to look up in `titleToSlug` (built from note *titles*, not paths). Lookup fails, it emits an "unknown wikilink" warning and leaves the malformed link untouched — so however the LLM extraction step originally produced a path-style link (likely a schema/prompt slip, not this function), this canonicalization pass has no path to actually repair it. ## Scope Before falling back to the title-lookup path, detect and strip a leading `wing:` or `wiki/` (or any known root prefix) from the bracket content, and try resolving the remainder as a path-style `wing/hall/slug` or `hall/slug` target directly (same resolution style the audit tooling in brain-gardener already uses per `knowledge/brain-graph-key-by-path-not-basename-slug.md`). Also worth checking `BuildPrompt`'s schema instructions — if the LLM is told exact link syntax and still produces prefixed forms, the schema/prompt may need tightening too. ## Scope (repair) Fix the 4 existing links to clean `hall/slug` form once the normalizer handles this case. ## Acceptance criteria - Regression test covering both prefix shapes resolving correctly. - The 4 links repaired.
mathias commented 2026-07-27 12:33:14 +00:00
Author
Owner
Copy Link
Copy Source

Fixed in hyperguild@6d014c1. canonicalizeContent now strips a known wing:/wiki/ root prefix and emits the clean path-style target directly when the title-lookup fails (path-style targets never match a title, so they always fell through to the unknown-wikilink warning before). Regression tests cover both prefix shapes; confirmed title-lookup still takes priority for normal display-name links.

Repaired the 4 known-broken links in mathias/brain@3bcae87 (confirmed all 4 targets exist before editing):

  • wing:homelab/failures/rootless-buildah-act-runner-run-containers-denied → homelab/failures/rootless-buildah-act-runner-run-containers-denied
  • wing:homelab/failures/act-runner-host-executor-tmp-persists → homelab/failures/act-runner-host-executor-tmp-persists
  • wiki/agentsquad/decisions/autoresearch-council-sibling-pipe → agentsquad/decisions/autoresearch-council-sibling-pipe
  • wiki/agentsquad/facts/llm-council-design-and-first-runs-2026-06-21 → agentsquad/facts/llm-council-design-and-first-runs-2026-06-21
Fixed in hyperguild@6d014c1. `canonicalizeContent` now strips a known `wing:`/`wiki/` root prefix and emits the clean path-style target directly when the title-lookup fails (path-style targets never match a title, so they always fell through to the unknown-wikilink warning before). Regression tests cover both prefix shapes; confirmed title-lookup still takes priority for normal display-name links. Repaired the 4 known-broken links in `mathias/brain@3bcae87` (confirmed all 4 targets exist before editing): - `wing:homelab/failures/rootless-buildah-act-runner-run-containers-denied` → `homelab/failures/rootless-buildah-act-runner-run-containers-denied` - `wing:homelab/failures/act-runner-host-executor-tmp-persists` → `homelab/failures/act-runner-host-executor-tmp-persists` - `wiki/agentsquad/decisions/autoresearch-council-sibling-pipe` → `agentsquad/decisions/autoresearch-council-sibling-pipe` - `wiki/agentsquad/facts/llm-council-design-and-first-runs-2026-06-21` → `agentsquad/facts/llm-council-design-and-first-runs-2026-06-21`
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
agent/ready
Meets the agent-ready contract; authorizes unattended execution
 blocked/client
Waiting on the client
 blocked/supplier
Waiting on a supplier
 blocked/upstream
Waiting on an upstream project or vendor
 risk/high
Irreversible, security-relevant, or control plane
 risk/low
Reversible, local, test-covered
 risk/medium
Touches shared state, deploys or data
 size/l
Too big - split it
 size/m
Under two days
 size/s
Under half a day
 size/xs
Under an hour
 status:dispatch
status:dispatched
status:review
status:reviewing
type/chore
Maintenance, no behaviour change
 type/decision
Human decision required; never agent/ready
 type/docs
Documentation only
 type/feat
New capability
 type/fix
Corrects broken behaviour
 type/ops
Infrastructure, deploy, runtime
 type/research
Investigate; output is a finding, not code
 type/test
Test coverage only
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate-bot mathias
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mathias/hyperguild#87