test: PR checks on a broken PR, left broken (do not merge) by marcleblanc2 · Pull Request #1920 · sourcegraph/docs

marcleblanc2 · 2026-09-11T15:09:30Z

Test PR to exercise the spell check, check-links, and check-redirects PR checks on one commit that breaks all three. This PR stays broken; #1921 applies the checks' suggested fixes and then fixes the rest by hand, for a side-by-side comparison.

Built on main plus the check PRs not merged yet, so their latest code runs here: #1880 (check-redirects), #1916 (check-links report format), #1919 (spell check inline-comment updates).

Replaces #1913. Do not merge.

Merge order for the PR-check stack

Trial-merged onto main in this order with no conflicts:

ci: Comment the Vercel build log on PRs whose build fails #1918 Vercel build log comment — independent; first so the other PRs' Vercel failures get a readable log
check-links: one suggestion per fix, synced with findings; one fact per line in reports #1916 check-links report format — adds dev/sync-review-comments.sh, which ci/redirects: Add Redirect check for PRs #1880 calls
ci/redirects: Add Redirect check for PRs #1880 redirect check — needs check-links: one suggestion per fix, synced with findings; one fact per line in reports #1916 merged first
spell check: update inline comments whose text changed #1919 spell check comment updates — independent
check-links: don't fail the generated-docs sync PR over absolute self-links it can't fix #1910 check-links, generated-docs sync PR — conflicts with check-links: one suggestion per fix, synced with findings; one fact per line in reports #1916 on dev/check-links.mjs; rebase after check-links: one suggestion per fix, synced with findings; one fact per line in reports #1916 merges

Squash-merge each, then rebase the next onto main.

#1920 (broken) and #1921 (fixed) are the example PRs that exercise every check; never merge, close them once the stack has landed.

dev/check-redirects.mjs checks every entry in src/data/redirects.ts: source shadows a page, source has a #fragment, duplicate source, /docs prefix, chained redirect, missing destination page or heading. The workflow compares against the merge base, so only redirects a PR breaks are reported, grouped by problem with the fix explained under each heading, and posts one suggested change per fixable entry the PR added (deleted again once the finding is gone). Not part of `npm run check`: main has hundreds of pre-existing findings. Squash of the check-redirects branch rebased onto main; the check-links commits it carried are already on main. Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

…ady names Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

…h (from the check-links PR) Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

… fact per line - Review comments: one suggested change per finding with a fix, no review body. Each starts with a marker so the workflow can delete suggestions for findings that are fixed and skip ones already posted. - Summary comment and review comments list line, link, problem, and fix on their own lines. - Absolute links to this site get their own section instead of Outbound. - Case-mismatch findings now carry a fix. - Wording: 'links on this site', 'these other pages', drop docs.sourcegraph.com; reproduce command matches package.json. Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

…roblem Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

… update comments whose text changed Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

vercel · 2026-09-11T15:09:36Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
sourcegraph-docs	Error		Sep 11, 2026 4:24pm UTC

github-actions · 2026-09-11T15:09:44Z

❌ This PR breaks 8 redirect(s)

Redirects are used so inbound traffic from external sources (links inside old versions of our product, bookmarks, search results, etc.) to old doc pages still reaches a relevant page.

A correct entry maps the old page path, exactly as the browser requests it, straight to a page that exists today, with an optional #heading that exists on the destination page:

{
	source: '/old/section/page',
	destination: '/new/section/page#heading-slug'
},

Each section below explains how to fix the entries listed under it.

Do not use redirects for broken internal links, internal links must be fixed properly to tame the tech debt snowball no one wants to deal with; the "Check links" PR check comment lists the links this PR broke, if any.

src/data/redirects.ts

Source overshadows a docs page that exists

Redirects take precedence over pages, so visitors to that page's URL are redirected away from it. Update or remove the redirect or the page to remove the conflict.

line 25

source: '/admin/tls_ssl',
destination: '/self-hosted/http-https-configuration'

Source has a #fragment, so this redirect can never match

Use the page path alone as the source. #fragments are processed in the browser, so browsers never send them to web servers.

If the redirect destination has a #fragment, it takes precedence, otherwise if the customer clicked a link which has a #fragment, it'll be kept and tried on the destination page.

line 17

source: '/old-ownership#anchor',
destination: '/code-ownership'

Fix: remove this entry

Source or destination starts with /docs

Write paths without the /docs prefix. The site removes /docs from the requested URL before matching sources, and adds it back in front of the destination, so a /docs/... source never matches and a /docs/... destination lands on /docs/docs/....

line 5

source: '/docs/old-prefixed',
destination: '/code-ownership'

Fix: change the source to /old-prefixed

Source already has an earlier entry, so this one is never used

Only the first entry for a source matches. Update that entry instead of adding another.

line 5916

source: '/admin/http_https_configuration',
destination: '/self-hosted/http-https-configuration'

Fix: remove this entry

Destination is another redirect

Chained redirects cost the customer's browser a round trip, slow down their page load time, and frustrate them. They also make the redirects file impossible to maintain, and make it too easy to create redirect loops. Change the rule's destination to the final destination.

line 9
```
source: '/old-chain',
destination: '/admin/http_https_configuration'
```
Fix: change the destination to /self-hosted/http-https-configuration

Destination page does not exist

Set the redirect destination to the page that replaced it, or remove the rule if there is no replacement page; visitors then get our fancy 404 page, with links they can click to find where they're trying to go, and the search bar.

line 13

source: '/old-ownership',
destination: '/code-ownershp'

line 715

source: '/user/markdown',
destination: '/admin/markdown'

Destination heading does not exist

Use the heading's correct anchor, or drop the #fragment to land the customer at the top of the page.

line 2647

source: '/batch_changes/how-tos/server_side_file_mounts',
destination: '/batch-changes/server-side#using-file-mounts-with-server-side-execution'

Fix: change the destination to /batch-changes/server-side

Reproduce locally with node dev/check-redirects.mjs

github-actions · 2026-09-11T15:09:45Z

+	{
+		source: "/docs/old-prefixed",
+		destination: "/code-ownership"
+	},


Problem: Source or destination starts with /docs
Fix: change the source to /old-prefixed

Suggested change

{

source: "/docs/old-prefixed",

destination: "/code-ownership"

},

{

source: '/old-prefixed',

destination: '/code-ownership'

},

github-actions · 2026-09-11T15:09:45Z

+	{
+		source: "/old-chain",
+		destination: "/admin/http_https_configuration"
+	},


Problem: Destination is another redirect
Fix: change the destination to /self-hosted/http-https-configuration

Suggested change

{

source: "/old-chain",

destination: "/admin/http_https_configuration"

},

{

source: '/old-chain',

destination: '/self-hosted/http-https-configuration'

},

github-actions · 2026-09-11T15:09:45Z

+	{
+		source: "/old-ownership#anchor",
+		destination: "/code-ownership"
+	},


Problem: Source has a #fragment, so this redirect can never match
Fix: remove this entry

Suggested change

{

source: "/old-ownership#anchor",

destination: "/code-ownership"

},

github-actions · 2026-09-11T15:09:45Z

+	{
+		source: "/admin/http_https_configuration",
+		destination: "/self-hosted/http-https-configuration"
+	}


Problem: Source already has an earlier entry, so this one is never used
Fix: remove this entry

Suggested change

{

source: "/admin/http_https_configuration",

destination: "/self-hosted/http-https-configuration"

}

github-actions · 2026-09-11T15:09:46Z

❌ This PR introduces 8 broken link(s)

Outbound

Your PR includes links to pages or anchors that do not exist.

docs/code-search/features.mdx

line 154
- Link: /code-search/no-such-page
- Problem: Page not found: "/code-search/no-such-page"
line 154
- Link: /code-search/features#no-such-heading
- Problem: Anchor "no-such-heading" not found in "/code-search/features"
line 154
- Link: /Code-Search/queries
- Problem: Path case mismatch: works on macOS, 404s on the Linux build
- Fix: /code-search/queries
line 152
- Link: https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md
- Problem: External link returns HTTP 404

Absolute links

Write links on this site as relative paths (/admin/config/site-config), not https://sourcegraph.com/docs/…: absolute links leave the preview deployment and local dev server, and hide moved pages behind redirects.

docs/code-search/features.mdx

line 152
- Link: https://sourcegraph.com/docs/admin/config/site-config
- Problem: Absolute link to this site
- Fix: /admin/config/site-config
line 152
- Link: //www.sourcegraph.com/docs/code-search/
- Problem: Absolute link to this site
- Fix: /code-search
line 152
- Link: http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2
- Problem: Absolute link to a moved page
- Fix: /self-hosted/http-https-configuration#sourcegraph-via-docker-compose-caddy-2

Inbound

A change your PR made broke inbound links from these other files. Please fix the inbound links in these other files.

docs/code-navigation/search-based-code-navigation.mdx

line 20
- Link: /code-search/features#symbol-search
- Problem: Anchor "symbol-search" not found in "/code-search/features"

Reproduce locally with pnpm check links --check-anchors --check-self-links (see dev/check-links.mjs).

Adding a redirect in src/data/redirects.ts does not satisfy this check, because it’s a workaround instead of a fix.

github-actions · 2026-09-11T15:09:47Z

 - When viewing a file or directory, press the `y` key to expand the URL to its canonical form (with the full 40-character Git commit SHA).
 - To share a link to multi-line range in a file, click on the starting line number and shift-click on the ending line number (in the left-hand gutter).
+
+Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).


Link: https://sourcegraph.com/docs/admin/config/site-config
Problem: Absolute link to this site
Fix: /admin/config/site-config

Suggested change

Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

Test links for the check-links workflow (will be reverted): [site config](/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

github-actions · 2026-09-11T15:09:47Z

 - When viewing a file or directory, press the `y` key to expand the URL to its canonical form (with the full 40-character Git commit SHA).
 - To share a link to multi-line range in a file, click on the starting line number and shift-click on the ending line number (in the left-hand gutter).
+
+Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).


Link: //www.sourcegraph.com/docs/code-search/
Problem: Absolute link to this site
Fix: /code-search

Suggested change

Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](/code-search), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

github-actions · 2026-09-11T15:09:47Z

 - When viewing a file or directory, press the `y` key to expand the URL to its canonical form (with the full 40-character Git commit SHA).
 - To share a link to multi-line range in a file, click on the starting line number and shift-click on the ending line number (in the left-hand gutter).
+
+Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).


Link: http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2
Problem: Absolute link to a moved page
Fix: /self-hosted/http-https-configuration#sourcegraph-via-docker-compose-caddy-2

Suggested change

Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](/self-hosted/http-https-configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

github-actions · 2026-09-11T15:09:47Z

+
+Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).
+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).


Link: /Code-Search/queries
Problem: Path case mismatch: works on macOS, 404s on the Linux build
Fix: /code-search/queries

Suggested change

More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).

More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/code-search/queries).

github-actions · 2026-09-11T15:09:50Z

⚠️ Spell check found 11 issue(s) in this PR

Only findings on lines added by this PR are shown.

docs/code-search/features.mdx

line 156, column 77
- indexs → indexes
line 156, column 90
- repositry → repository
line 156, column 100
- acros → arcos
line 156, column 128
- seach → search
line 156, column 146
- alwasy → always
line 156, column 169
- compatability → compatibility
line 156, column 192
- databse → database
line 156, column 215
- exector → executor
line 156, column 255
- contiainer → container

src/data/redirects.ts

line 14, column 23
- ownershp → ownership

cspell-allow-list.txt

line 602, column 1
- aardvark is out of alphabetical order: move it above acmeco on line 26.

Please correct the spelling, or add words which are correct to cspell-allow-list.txt.

Run npx cspell@10 --no-progress --dot '**/*' locally to check the full repository.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


indexs is not in the dictionary.

Did you mean indexes?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexes every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


repositry is not in the dictionary.

Did you mean repository?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repository acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


acros is not in the dictionary.

Did you mean arcos?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry arcos your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


seach is not in the dictionary.

Did you mean search?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so search results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


alwasy is not in the dictionary.

Did you mean always?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are always fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


databse is not in the dictionary.

Did you mean database?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the database layer, and the exector runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:51Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


exector is not in the dictionary.

Did you mean executor?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the executor runs each batch spec in its own contiainer.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:52Z

+
+More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).
+
+Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.


contiainer is not in the dictionary.

Did you mean container?

Suggested change

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own container.

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:52Z

+	},
+	{
+		source: "/old-ownership",
+		destination: "/code-ownershp"


ownershp is not in the dictionary.

Did you mean ownership?

Suggested change

destination: "/code-ownershp"

destination: "/code-ownership"

Please correct the spelling, or add the word to cspell-allow-list.txt if it is correct.

github-actions · 2026-09-11T15:09:52Z

 Zaporizhzhia
 Zoekt
 zoomable
+aardvark # test entry for the sorted-list check (will be reverted)


aardvark is out of alphabetical order: move it above acmeco on line 26.

Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9

…ving the old wording Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

Vercel shows build logs only to members of its team, so contributors saw a red X and a login wall. On vercel.deployment.error this posts the tail of the build log on the PR; on the next successful build the same comment is updated to say the failure is fixed. Fork PRs are skipped so the project-scoped Vercel token is never used on their behalf.

… (do not merge) Check links: three absolute self-links (one to a moved page) and a dead external link on one line; a missing page, missing heading, and wrong-case path on the next; and the "Symbol search" heading renamed to break the inbound anchor link from search-based-code-navigation.mdx. Spell check: nine misspellings on one line, five of them block-list words. Check redirects: one broken entry per category (shadowed page, #fragment source, /docs prefix, duplicate source, chain, missing page, missing heading). Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9

marcleblanc2 · 2026-09-11T16:27:10Z

❌ The Vercel build failed for this PR

Vercel only shows build logs to members of its team, so here is the end of the log.
Run npm run build locally to reproduce.

Build log

Running build in Cleveland, USA (East) – cle1
Build machine configuration: 4 cores, 8 GB
Cloning github.com/sourcegraph/docs (Branch: test-pr-checks-broken-2, Commit: 7333b3f)
Cloning completed: 1.111s
Restored build cache from previous deployment (HAcRus1q3izv8VuYaxAoNoaHLeYH)
Running "vercel build"
Vercel CLI 59.11.7
Error: Node.js version 20.x is deprecated. Deployments created on or after 2026-10-01 will fail to build. Please set "engines": { "node": "24.x" } in your `package.json` file to use Node.js 24.
Detected `pnpm-lock.yaml` version 9 generated by pnpm@10.x from package.json#packageManager pnpm@10.25.0
Installing dependencies...
Lockfile is up to date, resolution step is skipped
Already up to date

Done in 1.4s using pnpm v10.25.0
Detected Next.js version: 14.2.35
Running "pnpm run build"

> sourcegraph-docs@0.1.0 build /vercel/path0
> node dev/checks.mjs && node dev/generate-mermaid-icons.mjs && next build

🔍 Checking for dead links in MDX files...

❌ Found 2 dead link(s) in 1 file(s):


📄 docs/code-search/features.mdx
   Line 154: /code-search/no-such-page
   └─ Page not found: "/code-search/no-such-page"
   Line 154: /Code-Search/queries
   └─ Path case mismatch: works on macOS, 404s on the Linux build; use /code-search/queries
🔍 Checking for underscores in docs filenames...

❌ Found 1 path(s) with underscores:

   docs/admin/tls_ssl.mdx

   Please use hyphens (-) instead of underscores (_) in file and folder names.

🔍 Checking for raw <img> tags in MDX files...

✅ No raw <img> tags found in MDX files!

❌ Failed checks: links, filenames
 ELIFECYCLE  Command failed with exit code 1.
Error: Command "pnpm run build" exited with 1

marcleblanc2 · 2026-09-11T17:26:42Z

Moved to #1948 (branch renamed to marc/test-pr-checks-broken-2).

marcleblanc2 and others added 8 commits September 11, 2026 09:08

check-redirects: don't repeat the final destination the Fix line alre…

b2f29c2

…ady names Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

check-redirects: sync review comments with dev/sync-review-comments.s…

1508206

…h (from the check-links PR) Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

cspell: allow tostring, a jq builtin used in the workflows

e2f24a2

Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

check-links: don't repeat the link and fix inside the case-mismatch p…

f922886

…roblem Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

check-links: inbound section wording

41aae0f

Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

check-links: move review-comment sync to dev/sync-review-comments.sh;…

2407a71

… update comments whose text changed Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

marcleblanc2 mentioned this pull request Sep 11, 2026

test: PR checks on a broken PR, left broken (do not merge) #1913

Closed

github-actions Bot reviewed Sep 11, 2026

View reviewed changes

vercel Bot had a problem deploying to Preview September 11, 2026 15:09 Failure

github-actions Bot reviewed Sep 11, 2026

View reviewed changes

marcleblanc2 mentioned this pull request Sep 11, 2026

test: PR checks on a broken PR, then remediated (do not merge) #1921

Closed

marcleblanc2 and others added 2 commits September 11, 2026 09:12

cspell: allow argjson, a jq option used in dev/sync-review-comments.sh

9bc0896

Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9

spell check: update inline comments whose text changed instead of lea…

e9fe828

…ving the old wording Amp-Thread-ID: https://ampcode.com/threads/T-01a08fee-74b4-76dc-aaf9-d1245d68fdc9 Co-authored-by: Amp <amp@ampcode.com>

marcleblanc2 force-pushed the test-pr-checks-broken-2 branch from 595eda4 to d5c0ea6 Compare September 11, 2026 15:12

vercel Bot had a problem deploying to Preview September 11, 2026 15:13 Failure

marcleblanc2 and others added 2 commits September 11, 2026 10:24

marcleblanc2 force-pushed the test-pr-checks-broken-2 branch from d5c0ea6 to 7333b3f Compare September 11, 2026 16:24

vercel Bot had a problem deploying to Preview September 11, 2026 16:24 Failure

This was referenced Sep 11, 2026

ci: Comment the Vercel build log on PRs whose build fails #1918

Closed

check-links: one suggestion per fix, synced with findings; one fact per line in reports #1916

Open

ci/redirects: Add Redirect check for PRs #1880

Closed

marcleblanc2 closed this Sep 11, 2026

marcleblanc2 deleted the test-pr-checks-broken-2 branch September 11, 2026 17:27


		Test links for the check-links workflow (will be reverted): [site config](https://sourcegraph.com/docs/admin/config/site-config), [search](//www.sourcegraph.com/docs/code-search/), [moved page](http://docs.sourcegraph.com/admin/http_https_configuration#sourcegraph-via-docker-compose-caddy-2), [dead external](https://github.com/sourcegraph/docs/blob/main/this-file-does-not-exist.md), [live external](https://github.com/sourcegraph/docs).

		More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).


		More test links, none of which the check can fix for you: [missing page](/code-search/no-such-page), [missing heading](/code-search/features#no-such-heading), [wrong case](/Code-Search/queries).

		Test paragraph for the spell check workflow (will be reverted): Sourcegraph indexs every repositry acros your organization, so seach results are alwasy fresh. It keeps compatability with the databse layer, and the exector runs each batch spec in its own contiainer.

Uh oh!

Conversation

marcleblanc2 commented Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge order for the PR-check stack

Uh oh!

vercel Bot commented Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Sep 11, 2026

❌ This PR breaks 8 redirect(s)

Source overshadows a docs page that exists

Source has a #fragment, so this redirect can never match

Source or destination starts with /docs

Source already has an earlier entry, so this one is never used

Destination is another redirect

Destination page does not exist

Destination heading does not exist

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Sep 11, 2026

❌ This PR introduces 8 broken link(s)

Outbound

Absolute links

Inbound

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ Spell check found 11 issue(s) in this PR

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot Sep 11, 2026

Choose a reason for hiding this comment

marcleblanc2 commented Sep 11, 2026 •

edited

Loading

vercel Bot commented Sep 11, 2026 •

edited

Loading

github-actions Bot Sep 11, 2026 •

edited

Loading

github-actions Bot commented Sep 11, 2026 •

edited

Loading

github-actions Bot Sep 11, 2026 •

edited

Loading

marcleblanc2 commented Sep 11, 2026 •

edited

Loading