Skip to content

Comments

Git-based link-index for codex clone"#2739

Merged
reakaleek merged 7 commits intomainfrom
feature/codex-link-index
Feb 19, 2026
Merged

Git-based link-index for codex clone"#2739
reakaleek merged 7 commits intomainfrom
feature/codex-link-index

Conversation

@reakaleek
Copy link
Member

@reakaleek reakaleek commented Feb 18, 2026

Why:

The codex link index lives in the elastic/codex-link-index git repo, not S3. We need to clone it via SSH so it works with private repos and local git credentials.

What:

  • Add GitLinkIndexReader that clones elastic/codex-link-index into ~/.docs-builder/codex-link-index and reads the link index from there.
  • Make codex clone use the link index to resolve pinned commit SHAs (same idea as assembler clone).
  • Update CrossLinkFetcher to use ILinkIndexReader instead of hardcoded S3 URLs so it works with both S3 and git backends.

Result:

Codex clone uses the link index for reproducible builds, and developers can use their SSH setup to access the link index without S3 credentials.

Note

The git-based link index is not yet used for cross-link validation. That will be added in a follow-up.

@reakaleek reakaleek requested a review from a team as a code owner February 18, 2026 22:31
@reakaleek reakaleek requested review from Mpdreamz and cotti February 18, 2026 22:31
reakaleek and others added 2 commits February 19, 2026 10:21
…ne' may silently drop its earlier arguments'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@reakaleek reakaleek enabled auto-merge (squash) February 19, 2026 09:37
@reakaleek reakaleek disabled auto-merge February 19, 2026 09:40
await EnsureCloneAsync(cancellationToken);
if (Path.IsPathRooted(key))
throw new ArgumentException($"Repository key '{key}' must be a relative path.", nameof(key));
var linksPath = Path.Combine(CloneDirectory, key);
@reakaleek reakaleek enabled auto-merge (squash) February 19, 2026 09:56
@reakaleek reakaleek merged commit 221b86d into main Feb 19, 2026
30 checks passed
@reakaleek reakaleek deleted the feature/codex-link-index branch February 19, 2026 09:57
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.

2 participants