Skip to content

docs: stop repeating the section prefix in generated Markdown links - #1070

Merged
marekh19 merged 2 commits into
masterfrom
fix/section-prefix-in-generated-md-links
Sep 22, 2026
Merged

marekh19 merged 2 commits into
masterfrom
fix/section-prefix-in-generated-md-links

Conversation

@marekh19

Copy link
Copy Markdown
Contributor

Part of apify/apify-web#6665

@signalwire/docusaurus-plugin-llms-txt@1.2.2 prepends the site URL to link paths that already carry the Docusaurus baseUrl, so the generated Markdown points at https://docs.apify.com/api/client/python/api/client/python/..., which 404s. It only happens on sites served under a sub-path.

The fix is already upstream in 2.0.0-alpha.6, but it does not look like it will be published soon: npm latest is still 1.2.2 and nothing has shipped since 2.0.0-alpha.7 in November 2025. This patch applies that same fix locally. Remove it once a stable 2.x lands.

Built here: llms-full.txt goes from 4702 doubled links to 0, llms.txt stays at 352 correct links.

The same patch goes into apify-cli, apify-sdk-js, apify-sdk-python and apify-client-js.

@marekh19 marekh19 added the t-web Issues with this label are in the ownership of the web team. label Sep 22, 2026
@marekh19
marekh19 requested a review from webrdaniel September 22, 2026 07:36
@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.24%. Comparing base (efc009d) to head (0eb988c).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1070      +/-   ##
==========================================
- Coverage   95.25%   95.24%   -0.02%     
==========================================
  Files          59       59              
  Lines        5547     5547              
==========================================
- Hits         5284     5283       -1     
- Misses        263      264       +1     
Flag Coverage Δ
integration 92.06% <ø> (+0.01%) ⬆️
unit 87.39% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@marekh19
marekh19 marked this pull request as ready for review September 22, 2026 07:45
@marekh19
marekh19 requested a review from B4nan September 22, 2026 07:46
@marekh19 marekh19 changed the title fix: stop repeating the section prefix in generated Markdown links docs: stop repeating the section prefix in generated Markdown links Sep 22, 2026
@marekh19
marekh19 marked this pull request as draft September 22, 2026 07:52
@marekh19
marekh19 marked this pull request as ready for review September 22, 2026 07:53
@marekh19
marekh19 marked this pull request as draft September 22, 2026 09:22
@marekh19
marekh19 marked this pull request as ready for review September 22, 2026 11:26
@marekh19
marekh19 merged commit 396b997 into master Sep 22, 2026
31 checks passed
@marekh19
marekh19 deleted the fix/section-prefix-in-generated-md-links branch September 22, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-web Issues with this label are in the ownership of the web team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants