Skip to content

fix: fix how line anchor and file url generated. - #573

Open
GdoongMathew wants to merge 1 commit into
AsyncFuncAI:mainfrom
GdoongMathew:fix/wiki_markdown_link
Open

fix: fix how line anchor and file url generated.#573
GdoongMathew wants to merge 1 commit into
AsyncFuncAI:mainfrom
GdoongMathew:fix/wiki_markdown_link

Conversation

@GdoongMathew

Copy link
Copy Markdown
Collaborator

Summary

Since RepoUrlContext already determine the type of an repo, line anchor and file path generation should directly refer to that source instead.

@greptile-apps

greptile-apps Bot commented Aug 11, 2026

Copy link
Copy Markdown

Greptile Summary

The PR makes repository file URLs and line anchors use the repository type already established by RepoUrlContext, rather than inferring the hosting provider from the URL hostname.

  • Generates GitHub, GitLab, and Bitbucket file URLs from ctx.type.
  • Generates provider-specific line anchors from the same repository type.
  • Preserves bare paths for local repositories, missing URLs, and unknown types.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
api/services/wiki/content.py Replaces hostname parsing with the validated repository-type contract for file URL and line-anchor generation; no eligible follow-up defect remains.

Reviews (2): Last reviewed commit: "fix: fix how line anchor and file url ge..." | Re-trigger Greptile

@GdoongMathew
GdoongMathew marked this pull request as draft August 12, 2026 07:55
@GdoongMathew
GdoongMathew marked this pull request as ready for review August 12, 2026 10:17
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