fix(cli): accept canonical Forgejo pull request URLs

[escidmore]

Intent

Accept canonical Forgejo pull-request URLs as typed PR artifacts everywhere tasks-axi accepts PR links (fixes upstream issue #19: done --pr rejected any non-GitHub pull request URL). One authoritative strict PR URL seam - isPrUrl in src/pr-url.ts - shared by ordinary task link derivation (deriveLinks), done/add --pr validation, markdown task links, and public-followup pr_url deliverables. Preserve canonical GitHub https://github.com///pull/; add canonical Forgejo https://///pulls/. Strictly reject issue URLs, singular/plural route confusion, zero and leading-zero numbers, trailing slash, query/fragment, whitespace/control bytes, userinfo, ports, encoded separators, and malformed host/owner/repo segments; non-canonical PR-ish URLs in prose derive as doc links, never pr. Byte-exact rendering preserved; accurate help/docs (DONE_HELP Forgejo example, shared PR_URL_EXPECTED error text, README); TOP_HELP unchanged so the generated skills/tasks-axi/SKILL.md is intentionally untouched. Public-followup pr-merged pr_url uses the same seam under the same privacy-safe contract; the previous looser regex plus userinfo check was deliberately replaced by the strictly tighter isPrUrl. Scope is strictly typed links: no remote tracker backends, no API calls, no merge operations, no generalized forge client. This branch already passed a full pipeline run (PR #36, CI green); the new head applies two supervisor-decided review findings from that run: (1) typed-link whitespace normalization - the --pr value is now validated untrimmed (crud.ts requireTypedLinkUrl skips trim for pr kind; markdown.ts normalizeTypedLink validates the original url for pr kind), so whitespace-padded input is rejected consistently instead of silently normalized, with rejection tests at both the done --pr command layer and the store addLinks layer; (2) the literal NUL byte in test/pr-url.test.ts is now a unicode source escape so git treats the file as text, plus a genuine embedded-space rejection case. A third finding (strict-terminal-newline, claiming a regex dollar anchor matches before a trailing newline) was approved as factually incorrect for JavaScript: without the m flag the anchor matches only at end of input, and a trailing-newline rejection test passes. Update the existing PR #36; do not publish a package or merge the PR.

What Changed

• Add one strict PR URL classifier for canonical GitHub and Forgejo URLs, shared by prose link derivation, typed --pr values, markdown task links, and public-followup receipts.
• Reject whitespace-padded and other noncanonical PR-like inputs while classifying prose near-misses as document links.
• Document Forgejo usage and add regression coverage across URL, grammar, store, command, and public-followup paths.

Risk Assessment

✅ Low: The change is well-bounded, centralizes strict PR URL classification at the required shared seam, covers every specified caller, and introduces no material source or intent-conformance risks.

Testing

After correcting the missing-pnpm PATH setup with Corepack and one script-forwarding mistake, the focused validator, Markdown, state-command, and public-followup tests passed; real CLI evidence confirmed Forgejo acceptance, strict padded-input rejection, near-miss doc classification, persisted state, and updated help, then the temporary dependency directory was removed and the worktree confirmed unchanged. This is CLI-only, so screenshots were not applicable.

Evidence: Forgejo CLI transcript

\n$ tasks-axi add forgejo-demo --title Forgejo typed PR --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md --json
{
  "ok": true,
  "action": "add",
  "task": {
    "id": "forgejo-demo",
    "title": "Forgejo typed PR",
    "state": "queued",
    "kind": null,
    "repo": null,
    "priority": null,
    "created": "2026-08-08",
    "closed": null,
    "deps": [],
    "hold": null,
    "links": [],
    "body": null,
    "blocked": false,
    "blocked_by": [],
    "held": false
  }
}
\n$ tasks-axi done forgejo-demo --pr https://forgejo.example.com/eve/tasks/pulls/39 --no-prune --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md --json
{
  "ok": true,
  "action": "done",
  "pruned": 0,
  "task": {
    "id": "forgejo-demo",
    "title": "Forgejo typed PR https://forgejo.example.com/eve/tasks/pulls/39",
    "state": "done",
    "kind": null,
    "repo": null,
    "priority": null,
    "created": "2026-08-08",
    "closed": "2026-08-08",
    "deps": [],
    "hold": null,
    "links": [
      {
        "kind": "pr",
        "url": "https://forgejo.example.com/eve/tasks/pulls/39"
      }
    ],
    "body": null,
    "blocked": false,
    "blocked_by": [],
    "held": false
  }
}
\n$ tasks-axi show forgejo-demo --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md
task:
  id: forgejo-demo
  title: "Forgejo typed PR https://forgejo.example.com/eve/tasks/pulls/39"
  state: done
  blocked: no
  blocked_by: none
  held: no
  hold_reason: "-"
  hold_kind: "-"
  hold_until: "-"
  kind: task
  repo: "-"
  priority: "-"
  created: "-"
  closed: 2026-08-08
  deps: none
  links: "pr:https://forgejo.example.com/eve/tasks/pulls/39"
  body: ""
\n$ tasks-axi add padded-demo --title Reject padded PR --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md --json
{
  "ok": true,
  "action": "add",
  "task": {
    "id": "padded-demo",
    "title": "Reject padded PR",
    "state": "queued",
    "kind": null,
    "repo": null,
    "priority": null,
    "created": "2026-08-08",
    "closed": null,
    "deps": [],
    "hold": null,
    "links": [],
    "body": null,
    "blocked": false,
    "blocked_by": [],
    "held": false
  }
}
\n$ tasks-axi done padded-demo --pr " https://forgejo.example.com/eve/tasks/pulls/40 " --no-prune --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md --json
error: "Task pr link must be a canonical pull request URL: https://github.com/<owner>/<repo>/pull/<n> (GitHub) or https://<host>/<owner>/<repo>/pulls/<n> (Forgejo)"
code: VALIDATION_ERROR
exit: 2
\n$ tasks-axi add near-miss-demo --title Review https://forgejo.example.com/eve/tasks/pull/41 --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md --json
{
  "ok": true,
  "action": "add",
  "task": {
    "id": "near-miss-demo",
    "title": "Review https://forgejo.example.com/eve/tasks/pull/41",
    "state": "queued",
    "kind": null,
    "repo": null,
    "priority": null,
    "created": "2026-08-08",
    "closed": null,
    "deps": [],
    "hold": null,
    "links": [
      {
        "kind": "doc",
        "url": "https://forgejo.example.com/eve/tasks/pull/41"
      }
    ],
    "body": null,
    "blocked": false,
    "blocked_by": [],
    "held": false
  }
}
\n$ tasks-axi show near-miss-demo --file /var/folders/zf/93mvgfyd5_d7l2mz6zw_0fbw0000gn/T/no-mistakes-evidence/01KZG5JYKTSJBKW6BM0B565Q2S/forgejo-cli-backlog.md
task:
  id: near-miss-demo
  title: "Review https://forgejo.example.com/eve/tasks/pull/41"
  state: queued
  blocked: no
  blocked_by: none
  held: no
  hold_reason: "-"
  hold_kind: "-"
  hold_until: "-"
  kind: task
  repo: "-"
  priority: "-"
  created: 2026-08-08
  closed: "-"
  deps: none
  links: "doc:https://forgejo.example.com/eve/tasks/pull/41"
  body: ""
\n$ persisted markdown
# Backlog

## In flight
## Queued
- [ ] padded-demo - Reject padded PR (since 2026-08-08)
- [ ] near-miss-demo - Review https://forgejo.example.com/eve/tasks/pull/41 (since 2026-08-08)
## Done
- [x] forgejo-demo - Forgejo typed PR https://forgejo.example.com/eve/tasks/pulls/39 (merged 2026-08-08)

Evidence: Persisted Markdown backlog

# Backlog

## In flight
## Queued
- [ ] padded-demo - Reject padded PR (since 2026-08-08)
- [ ] near-miss-demo - Review https://forgejo.example.com/eve/tasks/pull/41 (since 2026-08-08)
## Done
- [x] forgejo-demo - Forgejo typed PR https://forgejo.example.com/eve/tasks/pulls/39 (merged 2026-08-08)

Evidence: DONE_HELP Forgejo example

tasks-axi done fj-task-q1 --pr https://forgejo.example.com/o/r/pulls/39

Pipeline

Updates from git push no-mistakes

✅ **intent** - passed

✅ No issues found.

✅ **Rebase** - passed

✅ No issues found.

✅ **Review** - passed

✅ No issues found.

✅ **Test** - passed

✅ No issues found.

• rtk git rev-parse HEAD
• rtk pnpm exec vitest run test/pr-url.test.ts test/backends/markdown-grammar.test.ts test/backends/markdown.test.ts test/commands/state.test.ts test/commands/public-followup.test.ts (pnpm was unavailable on PATH)
• rtk corepack pnpm --version
• rtk corepack pnpm install --frozen-lockfile
• rtk corepack pnpm exec vitest run test/pr-url.test.ts test/backends/markdown-grammar.test.ts test/backends/markdown.test.ts test/commands/state.test.ts test/commands/public-followup.test.ts
• rtk corepack pnpm dev -- done --help (incorrect script forwarding, then corrected)
• rtk corepack pnpm exec tsx bin/tasks-axi.ts done --help
• rtk corepack pnpm exec tsx bin/tasks-axi.ts show --help
• CLI evidence flow: add a task, complete it using --pr https://forgejo.example.com/eve/tasks/pulls/39, and inspect the resulting typed pr link and persisted Markdown
• CLI rejection flow: pass whitespace-padded Forgejo --pr input and verify VALIDATION_ERROR with exit 2
• CLI derivation flow: add prose containing https://forgejo.example.com/eve/tasks/pull/41 and verify it appears as a doc link, never pr
• rtk rm -rf /Users/host/.no-mistakes/worktrees/45059bee3575/01KZG5JYKTSJBKW6BM0B565Q2S/node_modules
• rtk git status --short

✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

[kunchenguid]

Thanks @escidmore - merged! Really appreciate the contribution.