Skip to content

docs(quickstart): send PDF failures to npm run doctor before reinstalling Chromium - #211

Open
vliggio wants to merge 1 commit into
career-ops-hq:mainfrom
vliggio:claude/issue-5722-career-ops-docs-b0e25e
Open

vliggio wants to merge 1 commit into
career-ops-hq:mainfrom
vliggio:claude/issue-5722-career-ops-docs-b0e25e

Conversation

@vliggio

@vliggio vliggio commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Hold until the next career-ops release. This describes doctor as it works after career-ops-hq/career-ops#5676. That PR is merged on main but not in any career-ops-v* release yet: the latest is career-ops-v1.37.0, and its doctor still says "not installed" for every launch failure. CONTRIBUTING.md asks docs to wait for a release, so merge this only after a career-ops-v* release includes #5676.

Closes career-ops-hq/career-ops#5721 and career-ops-hq/career-ops#5722 (5722 duplicates 5721).

The problem
In the Quick Start, the "The PDF did not generate" entry told everyone to run npx playwright install chromium again. That fixes only one cause, a missing browser. When system libraries are missing, or a sandbox stops the browser from starting, the install command finds Chromium already downloaded and changes nothing.

The change (content/docs/index.mdx)

  • Run npm run doctor first. Its Playwright check names the cause and prints the fix.
  • Run npx playwright install chromium only when doctor reports that Chromium is not installed.
  • When system libraries are missing, follow the dependency fix doctor prints for your system. The docs do not repeat the commands, because doctor's fix depends on the platform: install-deps with sudo on Debian or Ubuntu, the distro package manager on other Linux, an admin terminal or the Media Feature Pack on Windows, and the flake shell on NixOS.
  • For any other launch failure, reinstalling is not suggested. The reader is told to read doctor's error and check that their sandbox or container lets a browser start. This matches doctor's own text.
  • The Node.js version check stays as the final step.

Translations
ES, FR, DE, PT, IT and ZH are translated in the same PR. Each one was in sync before (a1ac86db5bf5c50e) and is now restamped to 7b12d8ccd43d60f2, so none is left out of date.

Not changed
set-up-playwright and claude-code already say to install Chromium only when it is missing, or when the install step was skipped.

npm run types:check and npm run build pass.

🤖 Generated with Claude Code

…ling Chromium

The troubleshooting entry told everyone to reinstall Chromium. That only
fixes a missing browser. Once career-ops-hq/career-ops#5676 ships, doctor
tells apart a missing browser, missing system libraries and other launch
failures, and prints the fix for each. The entry now sends readers to
doctor first and keeps the Node.js version check. All seven languages are
updated and restamped.

Refs career-ops-hq/career-ops#5721, career-ops-hq/career-ops#5722

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown

@vliggio is attempting to deploy a commit to the Career Ops OSS Program Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 58 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between c3caea2 and b4463fb.

📒 Files selected for processing: 7
  • content/docs/index.de.mdx
  • content/docs/index.es.mdx
  • content/docs/index.fr.mdx
  • content/docs/index.it.mdx
  • content/docs/index.mdx
  • content/docs/index.pt.mdx
  • content/docs/index.zh.mdx
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

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

@vliggio
vliggio marked this pull request as ready for review October 23, 2026 15:15
@vliggio
vliggio requested a review from santifer as a code owner October 23, 2026 15:15

This branch has not been deployed

No deployments
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.

docs: follow doctor diagnostics in PDF troubleshooting

1 participant