Skip to content

feat: 本文の GitHub の URL を、対応する生成ページへ向ける - #17

Merged
yktsnet merged 1 commit into
mainfrom
feat/github-links
Oct 14, 2026
Merged

yktsnet merged 1 commit into
mainfrom
feat/github-links

Conversation

@yktsnet

@yktsnet yktsnet commented Oct 14, 2026

Copy link
Copy Markdown
Owner

背景

リポの Markdown 本文や Issue の本文にある GitHub の URL は、生成物の中に対応するページがあっても外部 URL のままだった(Issue の本文では文字に落としていた)。オフラインや file:// で読むと辿れない。

変更

  • 走査対象のリポを owner/repo から引く索引(render.GitHubLinks)を build の前に作り、Markdown の描画に渡す
  • 内部へ向けるもの
    • リポのトップ → リポのページ
    • blob / tree → ref が HEAD のブランチ名と一致し、ページがあればそのファイル・ディレクトリのページ(アンカーは残す)
    • issues/N → 取り込み済みで、読んでいる文書と同じアカウントの Issue のページ
  • 対応しないもの(別の ref・HEAD に無いパス・未取り込みや別アカウントの Issue・PR・actions)は今までどおり
  • Issue のページの本文にも同じ規則をかける(_Sanitize の前に書き換える)
  • _owner_repo を repo.owner_repo へ移し、repo.branch を足した
  • 保証台帳 1・2 に項目を追加

確認

  • nix develop -c pytest -q: 34 passed。追加した2テストは、索引を渡さない・アカウントの判定を外す改変でそれぞれ落ちることを確かめた
  • 実データで build し直し、外部リンクのまま残った GitHub の URL が blob 549→126、issues 52→11、リポのトップ 141→84 に減った。残った blob は手元の HEAD に無いパスを指すもの

@yktsnet
yktsnet merged commit 4ef26ab into main Oct 14, 2026
3 checks passed
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