Skip to content

Redirects: resolve cross-repo to: in isolated CI - #3957

Merged
reakaleek merged 4 commits into
mainfrom
cursor/10e25a2f
Aug 28, 2026
Merged

Redirects: resolve cross-repo to: in isolated CI#3957
reakaleek merged 4 commits into
mainfrom
cursor/10e25a2f

Conversation

@reakaleek

Copy link
Copy Markdown
Member

Why

  • Isolated build --strict on content PRs did not resolve cross-repo redirects.yml to: values against the published link index.
  • A GitHub blob path such as kibana://docs/reference/advanced-settings.md passed required CI and broke assembler-preview on unrelated docs-builder PRs after merge.

What

  • Isolated builds now resolve :// redirect targets with the same TryResolve path assembler uses.
  • Writer docs state that the path is relative to the target repo docset.yml directory, and that diff validate only checks keys.

Notes

Made with Cursor

Isolated CI only URI-checked :// redirect targets, so a GitHub blob path could merge and fail assembler-preview later.

Co-authored-by: Cursor <cursoragent@cursor.com>
@reakaleek reakaleek added the bug label Aug 27, 2026
reakaleek and others added 3 commits August 28, 2026 01:37
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@reakaleek
reakaleek marked this pull request as ready for review August 28, 2026 05:45
@reakaleek
reakaleek requested a review from a team as a code owner August 28, 2026 05:45
@reakaleek
reakaleek requested a review from technige August 28, 2026 05:45

@Mpdreamz Mpdreamz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for plugging that gap!

@reakaleek
reakaleek merged commit 24795f6 into main Aug 28, 2026
25 checks passed
@reakaleek
reakaleek deleted the cursor/10e25a2f branch August 28, 2026 10:26
reakaleek added a commit that referenced this pull request Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Validate redirects.yml inputs

2 participants