Fix broken docs "Edit page" links via correct astro config editLink.baseUrl

[Copilot]

The docs "Edit page" anchor was rendering a malformed GitHub URL missing the verb segment between repo and branch (for example, .../aspire.dev/main/...), which leads to 404s. The root cause was that editLink.baseUrl in astro.config.mjs had the /edit/ verb stripped in an earlier commit, producing URLs like https://github.com/microsoft/aspire.dev/main/src/frontend/... instead of the correct https://github.com/microsoft/aspire.dev/edit/main/src/frontend/....

The fix is to ensure the editLink.baseUrl in astro.config.mjs retains the correct form with the action verb — no component-level workaround is needed.

• Root cause: editLink.baseUrl was incorrectly set to https://github.com/microsoft/aspire.dev/main/src/frontend/ (missing /edit/), causing Starlight to produce malformed edit URLs.
• Fix: The correct value https://github.com/microsoft/aspire.dev/edit/main/src/frontend/ is confirmed in place. The custom EditLink component continues to use Starlight's DefaultEditLink, relying on the config to produce valid URLs.

[Copilot]

Pull request overview

Fixes broken docs “Edit page” links by normalizing malformed GitHub URLs emitted via Astro.locals.starlightRoute.editUrl, ensuring they resolve to valid GitHub file URLs while leaving already-correct /edit/ and /blob/ links untouched.

Changes:

• Add GitHub URL normalization logic to the custom Starlight EditLink override.
• Replace the Starlight default EditLink component usage with an explicit <a> that uses the normalized URL.
• Add unit test coverage for malformed-vs-valid GitHub edit URLs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
src/frontend/src/components/starlight/EditLink.astro	Normalizes malformed GitHub URLs and renders the edit link using the corrected URL.
src/frontend/tests/unit/edit-link.vitest.test.ts	Adds regression tests for URL normalization and ensuring valid edit URLs are preserved.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[IEvangelist]

Hey @copilot, wasn't an easier fix to adjust the astro config itself instead?

[Copilot]

Hey @copilot, wasn't an easier fix to adjust the astro config itself instead?

Yes, you're right. The root cause was that editLink.baseUrl in astro.config.mjs had the /edit/ verb stripped at some point (it became ...aspire.dev/main/src/frontend/ instead of ...aspire.dev/edit/main/src/frontend/), which was already corrected in an earlier commit. The component-level URL normalization in our previous approach was unnecessary complexity.

I've reverted EditLink.astro to use DefaultEditLink as before and removed the test — the correct editLink.baseUrl config value (https://github.com/microsoft/aspire.dev/edit/main/src/frontend/) is the proper fix.

[mitchdenny]

This PR now appears to have no changes? Closing. Reopen if necessary.

[IEvangelist]

Hey @copilot, wasn't an easier fix to adjust the astro config itself instead?

Yes, you're right. The root cause was that editLink.baseUrl in astro.config.mjs had the /edit/ verb stripped at some point (it became ...aspire.dev/main/src/frontend/ instead of ...aspire.dev/edit/main/src/frontend/), which was already corrected in an earlier commit. The component-level URL normalization in our previous approach was unnecessary complexity.

I've reverted EditLink.astro to use DefaultEditLink as before and removed the test — the correct editLink.baseUrl config value (https://github.com/microsoft/aspire.dev/edit/main/src/frontend/) is the proper fix.

Please work on implementing the astro.config.mjs fix for this, as that was not done and this PR now has 0 files changed.

[Copilot]

Hey @copilot, wasn't an easier fix to adjust the astro config itself instead?

Yes, you're right. The root cause was that editLink.baseUrl in astro.config.mjs had the /edit/ verb strip...

The astro.config.mjs fix — restoring /edit/ in editLink.baseUrl — was already applied directly on main in commit 037d6137 (Feb 26, 2026):

 editLink: {
-  baseUrl: 'https://github.com/microsoft/aspire.dev/main/src/frontend/',
+  baseUrl: 'https://github.com/microsoft/aspire.dev/edit/main/src/frontend/',
 },

This PR branch was created after that commit landed, so the branch already starts with the correct config value — hence 0 files changed. The root cause is resolved on main. This PR can be closed.