Skip to content

fix: correct broken dev note links in recipe pages#407

Merged
dhruvnathawani merged 1 commit intomainfrom
dhruv/fix-links
Mar 13, 2026
Merged

fix: correct broken dev note links in recipe pages#407
dhruvnathawani merged 1 commit intomainfrom
dhruv/fix-links

Conversation

@dhruvnathawani
Copy link
Copy Markdown
Contributor

Summary

  • Recipe pages are 3 levels deep (recipes/category/recipe/), so links to dev notes need ../../../devnotes/ not ../../devnotes/
  • The extra level was missing, causing links to resolve to recipes/devnotes/ instead of devnotes/

Recipe pages are 3 levels deep (recipes/category/recipe/), so links
to dev notes need ../../../devnotes/ not ../../devnotes/. The extra
level was missing, causing links to resolve to recipes/devnotes/
instead of devnotes/.
@dhruvnathawani dhruvnathawani requested a review from a team as a code owner March 13, 2026 00:56
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 13, 2026

Greptile Summary

This PR fixes broken relative links to dev note pages in two recipe documentation files by correcting the path depth from ../../devnotes/ to ../../../devnotes/.

  • Recipe pages sit 3 levels deep under docs/ (docs/recipes/<category>/<recipe>.md), so navigating up to docs/ requires three ../ segments — the old paths only had two, causing links to resolve to the non-existent docs/recipes/devnotes/ directory.
  • The fix is consistent with the existing asset links in both files (e.g., ../../../assets/recipes/...), which already correctly use three levels.
  • Both docs/recipes/code_generation/enterprise_text_to_sql.md and docs/recipes/mcp_and_tooluse/search_agent.md are updated with the same one-character correction.

Confidence Score: 5/5

  • This PR is safe to merge — it is a minimal, targeted documentation fix with no code or logic changes.
  • The change is a one-level path depth correction in two markdown files. The fix is verifiably correct by tracing the directory structure (docs/recipes/<category>/<file>.md is 3 levels below docs/), and is consistent with the already-correct ../../../assets/ links present in both files. No functional code is touched.
  • No files require special attention.

Important Files Changed

Filename Overview
docs/recipes/code_generation/enterprise_text_to_sql.md Corrects the relative path depth for the dev note link from ../../devnotes/ to ../../../devnotes/, matching the 3-level-deep directory structure; consistent with the existing ../../../assets/ link already in the file.
docs/recipes/mcp_and_tooluse/search_agent.md Applies the same relative path correction for the dev note link from ../../devnotes/ to ../../../devnotes/, consistent with the file's depth and the existing ../../../assets/ link on line 9.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["docs/recipes/code_generation/enterprise_text_to_sql.md\ndocs/recipes/mcp_and_tooluse/search_agent.md"] -->|"../  (up 1)"| B["docs/recipes/&lt;category&gt;/"]
    B -->|"../../  (up 2)"| C["docs/recipes/"]
    C -->|"../../../  (up 3)"| D["docs/"]
    D -->|"devnotes/"| E["docs/devnotes/ ✅ CORRECT"]

    A -.->|"❌ OLD: ../../devnotes/"| F["docs/recipes/devnotes/ ❌ BROKEN"]

    style E fill:#22c55e,color:#fff
    style F fill:#ef4444,color:#fff
Loading

Last reviewed commit: 87f9039

@dhruvnathawani dhruvnathawani merged commit 0e868c9 into main Mar 13, 2026
47 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.

2 participants