Skip to content

docs: add a troubleshooting section for Console-connected repositories - #21

Merged
kristof-siket merged 2 commits into
prisma:mainfrom
quicksilversel:docs/troubleshooting
Nov 4, 2026
Merged

kristof-siket merged 2 commits into
prisma:mainfrom
quicksilversel:docs/troubleshooting

Conversation

@quicksilversel

Copy link
Copy Markdown
Contributor

Why

Two states around the Console's repository connection are hard to decode from the outside. The Console's setup job can fail at its install step (it resolves dependencies from scratch, so a peer-dependency conflict a committed lockfile would avoid can break it), and when it does, the pull request that would have added the Quick-start workflow never appears and nothing points back at this action. The Console then shows "Waiting for GitHub to start the first deployment", which reads as if pushing will fix it, but no push helps until a workflow with this action exists on the branch.

Change

A Troubleshooting section in the README, between Known limitations and Security, covering both states. Docs only.

Signed-off-by: quicksilversel <quicksilversel@gmail.com>
@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Summary by CodeRabbit

  • Documentation
    • Added troubleshooting guidance for Console setup jobs that do not open a pull request, including how to add the Quick-start workflow manually.
    • Clarified that the Console’s “Waiting for GitHub to start the first deployment” status remains until a workflow using the action runs on the default branch.

Walkthrough

The README adds troubleshooting guidance for setup-job failures caused by dependency resolution. It explains how to add the Quick-start workflow manually and states that the Console waiting status advances after a workflow containing the action deploys from the default branch.

Merge Risk: Minimal · up to c5055

The troubleshooting guidance adds no established merge-blocking risk and is ready to merge after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title "docs: add a troubleshooting section for Console-connected repositories" matches the main change. The PR adds a Troubleshooting section to the README that covers the Console setup job failin…
Description check Passed The description explains the problem and the change. It says the Console setup job can fail at its install step, and that the 'Waiting for GitHub to start the first deployment' status persists until a…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "docs: add a troubleshooting section for Console-connected repositories" matches the main change. The PR adds a Troubleshooting section to the README that covers the Console setup job failing at install and the 'Waiting for GitHub to start the first deployment' state. The title is specific, concise, and accurate.

Full details: Description check

Explanation

The description explains the problem and the change. It says the Console setup job can fail at its install step, and that the 'Waiting for GitHub to start the first deployment' status persists until a workflow using this action exists on the branch. It states that the change adds a Troubleshooting section to the README between Known limitations and Security and is docs only. The description is directly related to the changeset.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
Generate unit tests (beta)
  • Create a new PR
Simplify code
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 37e82218-eef3-4d03-be84-be3bf24208ab
Commits

Reviewing files that changed from the base of the PR and between 5e15880 and c505574.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@kristof-siket kristof-siket left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@quicksilversel great addition, thanks a lot! 🥇

@prisma-gizmo

prisma-gizmo Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

✅ Gizmo reviewed c505574 — posted 0 inline comment(s) this pass.

Open findings: none

Change walkthrough

Change walkthrough

This PR adds a two-bullet Troubleshooting section to README.md, placed between Known limitations and Security. It is documentation only — no code, action metadata, or tests are touched.

How it fits. The new section slots naturally into the README's existing structure: it leans on the Quick start workflow as the recovery path, complements Known limitations (install detection covering npm/bun lockfiles explains why the setup job's from-scratch install can fail on peer-dependency conflicts), and stays consistent with the Security section's description of the OIDC exchange matching the repository against the workspace connection. The claims about Console-side behavior (setup job, waiting state) are external to this repository and unverifiable from the code, but they do not contradict anything the README already documents.

@prisma-gizmo prisma-gizmo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

New findings: none · trace

@prisma-gizmo prisma-gizmo Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No Gizmo finding is open and the head commit has been reviewed. Approving.

@kristof-siket
kristof-siket merged commit 8673d42 into prisma:main Nov 4, 2026
3 checks 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.

2 participants