-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[docs]: update act reference with preferred model name formatting
#1409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs]: update act reference with preferred model name formatting
#1409
Conversation
|
| model: { | ||
| modelName: "gemini-2.5-flash", | ||
| apiKey: process.env.GOOGLE_API_KEY, | ||
| modelName: "google/gemini-2.5-flash", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming we want the google/ prefix here, like the example above it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes nice catch
Greptile OverviewGreptile SummaryThis PR updates v3 documentation to use the correct environment variable names for Google/Gemini API keys. The migration from v2 to v3 changed the recommended environment variable from Changes made:
All changes are consistent with the actual implementation in Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant Dev as Developer
participant Docs as V3 Documentation
participant Code as Stagehand Code
participant Env as Environment Variables
Note over Dev,Env: Before PR: Inconsistent Variable Names
Dev->>Docs: Reads v3 docs
Docs-->>Dev: Use GOOGLE_API_KEY
Dev->>Env: Sets GOOGLE_API_KEY
Dev->>Code: Initializes Stagehand
Code->>Env: Checks GEMINI_API_KEY
Code->>Env: Checks GOOGLE_GENERATIVE_AI_API_KEY
Code-->>Dev: ❌ Missing API key error
Note over Dev,Env: After PR: Aligned Documentation
Dev->>Docs: Reads v3 docs
Docs-->>Dev: Use GEMINI_API_KEY
Dev->>Env: Sets GEMINI_API_KEY
Dev->>Code: Initializes Stagehand
Code->>Env: Checks GEMINI_API_KEY
Code-->>Dev: ✅ Successfully initialized
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 files reviewed, no comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 4 files
act reference with preferred model name formatting
why
actreference to use"provider/model-name"formatting