Skip to content

[codex] fix broken internal link paths#643

Merged
riderx merged 2 commits into
mainfrom
codex/fix-broken-link-paths
May 6, 2026
Merged

[codex] fix broken internal link paths#643
riderx merged 2 commits into
mainfrom
codex/fix-broken-link-paths

Conversation

@riderx
Copy link
Copy Markdown
Member

@riderx riderx commented May 6, 2026

Summary

  • fix CSV-listed broken internal plugin links to use the repo-backed plugin page slugs
  • update the plugin README generator so public plugin docs URLs use GitHub repo slugs instead of display titles
  • point support and social-login docs links at valid destinations

Validation

  • bunx prettier --write (Markdown/MDX ignored by repo config)
  • bun run check

Summary by CodeRabbit

  • Documentation
    • Updated plugin documentation links across blog articles and guides to reference Capacitor-specific plugin versions.
    • Corrected internal documentation paths to use absolute URLs for better link consistency.
    • Updated support URL references in documentation.
    • Enhanced plugin documentation generation to improve link accuracy.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 6, 2026

Warning

Rate limit exceeded

@riderx has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 10 minutes and 15 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b95ab632-cc19-40b7-b176-02b30a2d9420

📥 Commits

Reviewing files that changed from the base of the PR and between 24a6a85 and 53a6b9d.

📒 Files selected for processing (7)
  • apps/web/src/content/blog/en/biometric-authentication-in-capacitor-apps.md
  • apps/web/src/content/blog/en/capacitor-live-updates-staying-compliant-with-apple.md
  • apps/web/src/content/blog/en/development-vs-production-key-differences-in-capacitor-apps.md
  • apps/web/src/content/blog/en/how-native-bridge-works-in-capacitor-android-apps.md
  • apps/web/src/content/blog/en/installing-capacitor-cli-step-by-step-guide.md
  • apps/web/src/content/blog/en/ssl-pinning-for-capacitor-apps.md
  • scripts/generate-plugins-readme.ts
📝 Walkthrough

Walkthrough

This PR standardizes plugin documentation links across docs and blog content by renaming plugin references from generic names (e.g., ivs-player) to namespaced versions (e.g., capacitor-ivs-player) and updating path formats. A helper function in the plugin README generation script now derives documentation URLs from plugin hrefs using slug extraction.

Changes

Plugin Documentation Link Standardization

Layer / File(s) Summary
URL Pattern Updates
apps/docs/src/content/docs/docs/plugins/social-login/firebase/google/*.mdx, apps/docs/src/content/docs/docs/webapp/organization-roles.mdx
Documentation links updated to use absolute paths and corrected support domain (capgo.app/supportsupport.capgo.app).
Blog Content Link Corrections
apps/web/src/content/blog/en/*.md
23 blog articles updated with renamed plugin references: ivs-playercapacitor-ivs-player, native-audiocapacitor-native-audio, native-purchasescapacitor-native-purchases, camera-previewcapacitor-camera-preview.
Plugin URL Generation Logic
scripts/generate-plugins-readme.ts
New getPluginSlug() helper function extracts URL-safe slugs from plugin hrefs; plugin cards now derive docs URLs from href-based slugs instead of title-based derivation. Updater plugin docs URL updated to capacitor-updater.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🐰 Links all tidy and neat,
Plugins renamed—refactor complete!
From ivs-player old,
To capacitor gold,
Our docs now have rhythm and beat! 🎵

Possibly related PRs

  • Cap-go/website#597: Updates plugin page quick-link logic to fall back to docs base path, related to how plugin docs links are resolved
  • Cap-go/website#596: Modifies plugin page docsHref/npmHref derivation and CTA behavior, overlapping with plugin URL standardization
  • Cap-go/website#593: Resolves per-plugin docs slugs and uses them in plugin page rendering, directly related to slug-based URL generation
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[codex] fix broken internal link paths' directly summarizes the main change in the PR: fixing broken/incorrect internal links across documentation and blog files by updating them to use correct plugin repository slugs and valid paths.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-broken-link-paths

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 7

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@apps/web/src/content/blog/en/biometric-authentication-in-capacitor-apps.md`:
- Line 56: The link in the intro sentence currently targets the unrelated
"capacitor-ivs-player" page which is misleading; update the link target text
used in the sentence (the bracketed link for "Capacitor app") to point to a
biometric-relevant or generic Capacitor setup page (for example Capacitor's
official docs or a biometric plugin guide) so readers land on relevant setup
info—locate the link in the paragraph containing "To enable biometric
authentication in your [Capacitor
app](https://capgo.app/plugins/capacitor-ivs-player/)" and replace the URL with
the appropriate biometric or generic Capacitor documentation URL.

In
`@apps/web/src/content/blog/en/capacitor-live-updates-staying-compliant-with-apple.md`:
- Line 90: The anchor in the intro currently links to the capacitor-ivs-player
page but should point to a live-updates/updater-related page; update the URL in
the string "Capacitor app" (the existing link target
"https://capgo.app/plugins/capacitor-ivs-player/") to the appropriate
live-updates/updater documentation page so the intro directs readers to
live-update implementation guidance.

In
`@apps/web/src/content/blog/en/development-vs-production-key-differences-in-capacitor-apps.md`:
- Line 154: Replace the incorrect deployment link
"https://capgo.app/plugins/capacitor-ivs-player/" used in the sentence starting
"Deploying a [Capacitor app]..." with a more appropriate Capacitor
deployment/update or general Capacitor resource (for example the official
Capacitor docs or a Capacitor deploy/update guide); locate the markdown link
text "[Capacitor app]" and update its URL to the chosen relevant resource so the
sentence points to a deployment-focused reference.

In
`@apps/web/src/content/blog/en/how-native-bridge-works-in-capacitor-android-apps.md`:
- Line 102: The "Capacitor Version" table row currently links to the wrong
resource (the capgo plugin page). Update the link target for the "[Capacitor
Version]" markdown link so it points to the official Capacitor framework
documentation (e.g. https://capacitorjs.com/docs) or the appropriate Capacitor
version docs, replacing the existing capgo.app/plugins/capacitor-ivs-player URL
so the row accurately references Capacitor framework/version information.

In `@apps/web/src/content/blog/en/installing-capacitor-cli-step-by-step-guide.md`:
- Line 115: The link target after "check the [Capacitor version]" points to a
plugin page instead of the Capacitor CLI/version docs; update the target URL to
the official Capacitor CLI/version documentation (for example the Capacitor
docs/CLI page) so the anchor "Capacitor version" directs readers to the correct
Capacitor CLI/version docs rather than the plugin page.

In `@apps/web/src/content/blog/en/ssl-pinning-for-capacitor-apps.md`:
- Line 57: Replace the misleading capacitor-ivs-player link text/URL in the
sentence that currently contains "capgo.app/plugins/capacitor-ivs-player/" with
a relevant SSL pinning or Capacitor setup resource (e.g., the Capacitor
Community SSL Pinning plugin or an OWASP/TLS pinning guide); update both the
anchor text and href so the link points to an SSL-pinning specific guide and the
visible text reflects that (search for the string
"capgo.app/plugins/capacitor-ivs-player" to locate the exact place to edit).

In `@scripts/generate-plugins-readme.ts`:
- Around line 67-70: getPluginSlug currently returns the final URL segment which
breaks for GitHub URLs with subpaths (e.g., /tree/main/packages/...), so change
getPluginSlug to parse the href, strip trailing slash, and if the host contains
"github.com" extract the repository name (the segment immediately after the
owner segment) as the slug (normalized to lowercase); otherwise fall back to the
existing last-segment behavior—use the existing normalizedHref variable and
update getPluginSlug to split the path into segments, pick segments[1] for repo
when github, and substring fallback for non-github URLs.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2a7ef9b3-76ec-4622-86b7-5211cb03e57c

📥 Commits

Reviewing files that changed from the base of the PR and between e72cc11 and 24a6a85.

📒 Files selected for processing (29)
  • apps/docs/src/content/docs/docs/plugins/social-login/firebase/google/android.mdx
  • apps/docs/src/content/docs/docs/plugins/social-login/firebase/google/ios.mdx
  • apps/docs/src/content/docs/docs/webapp/organization-roles.mdx
  • apps/web/public/plugins-readme.md
  • apps/web/src/content/blog/en/5-steps-to-implement-oauth2-in-capacitor-apps.md
  • apps/web/src/content/blog/en/biometric-authentication-in-capacitor-apps.md
  • apps/web/src/content/blog/en/capacitor-app-initialization-step-by-step-guide.md
  • apps/web/src/content/blog/en/capacitor-apps-and-russias-data-laws-compliance-tips.md
  • apps/web/src/content/blog/en/capacitor-community-forums-for-plugin-developers.md
  • apps/web/src/content/blog/en/capacitor-live-updates-staying-compliant-with-apple.md
  • apps/web/src/content/blog/en/capacitor-native-bridge-android-plugin-basics.md
  • apps/web/src/content/blog/en/capacitor-ota-updates-cloud-hosting-options-compared.md
  • apps/web/src/content/blog/en/capacitor-plugins-what-you-need-to-know.md
  • apps/web/src/content/blog/en/development-vs-production-key-differences-in-capacitor-apps.md
  • apps/web/src/content/blog/en/how-background-tasks-work-in-capacitor.md
  • apps/web/src/content/blog/en/how-capacitor-bridges-web-and-native-code.md
  • apps/web/src/content/blog/en/how-native-bridge-works-in-capacitor-android-apps.md
  • apps/web/src/content/blog/en/how-to-add-geolocation-targeting-to-ota-updates.md
  • apps/web/src/content/blog/en/implementing-native-bridge-for-ios-in-capacitor.md
  • apps/web/src/content/blog/en/installing-capacitor-cli-step-by-step-guide.md
  • apps/web/src/content/blog/en/server-setup-for-capacitor-ota-updates.md
  • apps/web/src/content/blog/en/set-up-performance-monitoring-in-capacitor.md
  • apps/web/src/content/blog/en/setting-up-cicd-for-capacitor-apps.md
  • apps/web/src/content/blog/en/ssl-pinning-for-capacitor-apps.md
  • apps/web/src/content/blog/en/top-api-security-standards-for-app-store-compliance.md
  • apps/web/src/content/blog/en/top-tools-for-debugging-ota-updates-in-capacitor.md
  • apps/web/src/content/blog/en/ultimate-guide-to-capacitor-plugin-development.md
  • apps/web/src/content/blog/en/usage-frequency-segmentation-for-app-updates.md
  • scripts/generate-plugins-readme.ts

Comment thread apps/web/src/content/blog/en/biometric-authentication-in-capacitor-apps.md Outdated
Comment thread apps/web/src/content/blog/en/how-native-bridge-works-in-capacitor-android-apps.md Outdated
Comment thread apps/web/src/content/blog/en/installing-capacitor-cli-step-by-step-guide.md Outdated
Comment thread apps/web/src/content/blog/en/ssl-pinning-for-capacitor-apps.md Outdated
Comment thread scripts/generate-plugins-readme.ts
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@riderx
Copy link
Copy Markdown
Member Author

riderx commented May 6, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 6, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@riderx riderx merged commit 31e392f into main May 6, 2026
10 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