docs: handle optional provider keys in setup check (bounty claim) - #68
Open
MaxKerkula wants to merge 1 commit into
Open
docs: handle optional provider keys in setup check (bounty claim)#68MaxKerkula wants to merge 1 commit into
MaxKerkula wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The quick setup check currently raises
TypeErrorwhenOPENAI_API_KEYis absent, including an Anthropic-only setup. It also prints part of an OpenAI key when one is present.This change reports whether each of the three documented provider keys has a nonempty value. It prints no key characters. The text explains that presence does not establish validity, and that unused providers can report
False.Validation: the exact Python block passes 54 isolated cases covering absent, empty, and synthetic nonempty values for all three providers, with and without a workspace value. The baseline fails in the Anthropic-only case. No real credentials or model API calls were used.
git diff --checkpasses. This patch changes no links.Link-check correction: CLI 4.2.876 reported 232 links in 70 files on native Windows for both trees. That output must not be treated as 232 established documentation defects. A fresh comparison on the same baseline checkout reports no broken links with Linux Node in WSL. An isolated fixture reproduces a Windows false positive for an existing nested page; the Linux check passes the valid fixture and fails its intentionally missing-page control. This is consistent with Mintlify issue #5680. Visual site rendering and external-link availability are separate checks.
Bounty claim: I request the Bronze tier under the documentation bounty program. This is one focused correction under 100 words. Please confirm eligibility and the award after review and merge. PayPal is my preferred payout method; payment details will be provided privately through a verified channel. No award or payment is assumed.
This contribution was prepared with AI assistance and checked locally.
Related: #65 proposed a similar correction and was closed, unmerged, by its author. The defect remains in the current base. These checks used the installed
python-dotenvpackage with temporary synthetic.envfiles.