ci: rewrite git SSH to HTTPS so Dependabot e2e can install - #552
Conversation
Dependabot lockfiles resolve @electron/node-gyp as git@github.com:. The setup job already rewrites that to HTTPS; e2e and commitlint did not, so those checks died in ~20s with Permission denied (publickey). Closes #544 Co-authored-by: Tomás Maritano <tomymaritano@gmail.com>
|
Warning Review limit reached
Next review available in: 23 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🎉 This PR is included in version 0.16.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
## Summary Documents the MCP / other-AI plan in a new folder `docs/ai/` (plan only, no product code). - `docs/ai/README.md` index - `docs/ai/mcp-plan.md` Inkdrop shape: local HTTP after AuthGate + one MCP; Grok Bot / Claude / Cursor share it - `docs/ai/updates.md` 22 Aug decisions (AuthGate stays, #541/#552, Project 3, no CouchDB, mobile Later) Does not implement MCP HTTP. Existing `packages/mcp-server` stays as the current stdio path. ## Test plan - [ ] Read the three files on the PR - [ ] Confirm no app/CI files changed
Summary
Dependabot PRs fail the required
e2echeck in ~20s duringpnpm install, not during Playwright. Automerge is already enabled; it cannot land while e2e is red.Exact failure (PR #482 job 96541625973, also #464 / #456 / #462 and
commitlinton #464):This is not missing repo secrets. Dependabot-regenerated lockfiles resolve
@electron/node-gypas a git dep whoseresolution.repois SSH:pnpm clones
resolution.repoover SSH. GitHub-hosted runners have no deploy key.The
setupjob already rewrites SSH to HTTPS and therefore succeeds on the same PRs.e2eandcommitlintdid a freshpnpm installwithout that rewrite.developitself is green after #541 — its lockfile uses the HTTPS tarball, so e2e never hits the SSH clone.Type of Change
Fix
Add the same
git config --global 'url.https://github.com/.insteadOf' 'git@github.com:'step thatsetup/release/build/docsalready use:.github/workflows/ci.ymle2ejob — unblocks the required e2e check.github/workflows/pr-title.yml— same SSH death on Dependabot lockfilesExisting Dependabot PRs (#482, #464, #462, #460, #459, #456, #451, #431, #430, #248) should go green after rebase onto this
developchange (the merge commit picks up the workflow).Related Issues
Closes #544
Checklist
pnpm test) — workflow-only change; no product codepnpm build) — not applicabledevelopbranch (notmain)