Skip to content

ci: authenticate private screenpipe-fork git dep in the Rust job#331

Merged
Akarsh-Hegde merged 1 commit into
pre-mainfrom
fix/ci-private-fork-auth
Jun 24, 2026
Merged

ci: authenticate private screenpipe-fork git dep in the Rust job#331
Akarsh-Hegde merged 1 commit into
pre-mainfrom
fix/ci-private-fork-auth

Conversation

@Akarsh-Hegde

Copy link
Copy Markdown
Member

Problem

CI on #330 (pre-mainmain) fails at the Rust job:

fatal: could not read Username for 'https://github.com': No such device or address
error: failed to get `screenpipe-a11y` as a dependency of package `meridian-tray`
  ... Unable to update https://github.com/Meridiona/screenpipe-fork?rev=d6d2b318...

ci.yml's rust job runs cargo fmt/clippy/test over the whole workspace. That resolves the tray's screenpipe-screen / screenpipe-a11y deps — a git dependency on the private Meridiona/screenpipe-fork — even though they're optional / capture-gated and never compiled here. With only the repo-scoped GITHUB_TOKEN, cargo's git fetch of the fork can't authenticate, so the job dies before clippy even runs.

Fix

Port the exact mechanism release.yml and release-staging.yml already use (those jobs build the tray against the fork and pass):

No source/build changes; CI-only. Merging this into pre-main re-triggers #330's CI, which should then go green.

🤖 Generated with Claude Code

The CI Rust job (fmt/clippy/test) resolves the whole workspace, including
the tray's optional capture deps (screenpipe-screen/-a11y) which are a git
dependency on the PRIVATE Meridiona/screenpipe-fork. With only the repo-scoped
GITHUB_TOKEN, cargo's `git fetch` of the fork fails ("could not read Username
for github.com"), so clippy/test never start.

Port the exact mechanism release.yml / release-staging.yml already use:
- CARGO_NET_GIT_FETCH_WITH_CLI=true so cargo honors url.insteadOf (libgit2 won't)
- a git config step rewriting the fork URL to embed secrets.GH_TOKEN

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SKdh3tKWLZhtQ9FCfA5RYH
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 5a083609-879d-406e-a1ab-fd414edbdeef

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-private-fork-auth

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

@Akarsh-Hegde Akarsh-Hegde merged commit 2161c98 into pre-main Jun 24, 2026
1 check 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