docs(integrations): fix GitHub personal-token instructions to not ask for a password - #46
Conversation
… for a password The "Personal token" auth section said "Provide username and password." GitHub has not accepted account passwords for this kind of integration since 2021 — PATs are mandatory. Every other token-based provider page in this docs set says "paste the generated token"; a user following this literally would enter their real GitHub password and fail. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Follow-up to the previous commit on this branch. Checked the actual
wizard screenshot (wz_step2_token.png): the real dialog has a single
"Personal token" field and no username field at all, matching
GitLab's equivalent step. The prior fix ("Provide your GitHub
username and the personal access token...") invented a username
field that doesn't exist in the UI. Corrected to match what the
screenshot actually shows and the phrasing convention used by every
other token-based provider page ("paste generated token").
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Confirmed both ways. I opened The server agrees — } else if (auth.mode === 'token') {
clientOptions.auth = auth.token.access_token;The self-correction called out in the description was the right instinct — the intermediate "username + token" wording would have been wrong too. Final wording matches both the screenshot and the convention on the other token-based provider pages. LGTM. 🤖 Generated with Claude Code |
krassx
left a comment
There was a problem hiding this comment.
Validated against source (see review comment above). Approving.
Summary
docs/integrations/providers/github/index.md, "Personal token" section, said: "Provide username and password."wz_step2_token.png) — the real dialog has a single "Personal token" field, no username field at all, same pattern as GitLab's equivalent step.Changes
Note: an earlier commit on this branch fixed the "password" wording but incorrectly added a username field that doesn't exist in the actual UI. That was caught during a re-verification pass and corrected in the second commit — the diff now reflects the final, screenshot-verified wording.
Test plan
npx cspellon the changed file — 0 issueswz_step2_token.pngscreenshot that the wizard step has only a token field🤖 Generated with Claude Code
https://claude.ai/code/session_01QnXDhVitCVmMNafPRvKUat