fix(cli): import BIOLM_PUBLIC_CLIENT_ID for biolm login - #9
Merged
Conversation
Without this import, `biolm login` crashed with NameError before OAuth could start. Co-authored-by: Cursor <cursoragent@cursor.com>
Invoke the login command via CliRunner so a missing BIOLM_PUBLIC_CLIENT_ID import fails in CI instead of at user runtime. Co-authored-by: Cursor <cursoragent@cursor.com>
Schema lookups now always hit the hosted platform API without credentials, so hub gateways and CI API tokens no longer break batch-size discovery. Also update the live schema test to assert maxItems instead of removed throttle_rate metadata. Co-authored-by: Cursor <cursoragent@cursor.com>
Split tests into smoke/unit/integration/live tiers so PR CI runs fast offline checks on Python 3.11 only, while live API and full-matrix runs move to nightly and main-branch compat jobs. Add path-based integration selection and gate live tests behind --run-live to avoid accidental network calls when tokens are set.
Provide a dummy BIOLM_TOKEN for smoke/unit/integration runs and add PyYAML to dev requirements so integration test selection can load test-areas.yml. Co-authored-by: Cursor <cursoragent@cursor.com>
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
1 participant
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.
Summary
biolm logincrashing withNameError: name 'BIOLM_PUBLIC_CLIENT_ID' is not definedbiolm.core.constso OAuth login can startTest plan
biolm loginstarts OAuth flow (browser opens, authorize URL printed)biolm statusshows valid credentials at~/.biolm/credentialsMade with Cursor