Skip to content

chore: default to gemini-3.1-pro-preview + gitignore internal docs - #22

Merged
th0rz05 merged 1 commit into
mainfrom
pr3-chore-model-gitignore
Jun 2, 2026
Merged

chore: default to gemini-3.1-pro-preview + gitignore internal docs#22
th0rz05 merged 1 commit into
mainfrom
pr3-chore-model-gitignore

Conversation

@th0rz05

@th0rz05 th0rz05 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Bump the structured-output default model to gemini-3.1-pro-preview.
  • Add .gitignore rules for .DS_Store and the local-only docs/claude_docs/ scratch directory.

Third of 3 small PRs.

Test plan

  • PYTHONPATH=src pytest -q (33 passed)
  • ruff check clean on changed files

Bumps the structured-output default model to gemini-3.1-pro-preview and
adds .gitignore rules for .DS_Store and the local-only docs/claude_docs/
scratch directory so internal debug notes are never committed.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 2, 2026 14:30
@th0rz05
th0rz05 merged commit 7b50349 into main Jun 2, 2026
@th0rz05
th0rz05 deleted the pr3-chore-model-gitignore branch June 2, 2026 14:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the campaign-setup agent’s structured-output default model to gemini-3.1-pro-preview, and adds local-only ignore rules to keep internal scratch docs and macOS metadata out of the repo.

Changes:

  • Bump campaign_setup default model from gemini-2.5-flash to gemini-3.1-pro-preview.
  • Adjust node-factory tests to assert init_chat_model is called with the resolved default model name.
  • Add .gitignore entries for .DS_Store and docs/claude_docs/.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/yieldagent/agents/campaign_setup/nodes.py Updates DEFAULT_MODEL and (needs) aligned documentation around provider resolution behavior.
tests/agents/test_nodes.py Updates tests to assert the default path uses _resolve_model_name(DEFAULT_MODEL).
.gitignore Ignores macOS .DS_Store and local-only docs/claude_docs/ scratch docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 16 to 20
# Provider is inferred from the model name by init_chat_model:
# gemini-* -> google_genai (requires GOOGLE_API_KEY)
# claude-* -> anthropic (requires ANTHROPIC_API_KEY)
# gpt-* -> openai (requires OPENAI_API_KEY)
# A bare gemini-* is rewritten to google_genai:gemini-* by _resolve_model_name
# (Google AI Studio, requires GOOGLE_API_KEY) — see that function for why.
# Override at the CLI via --model, or pass an explicit "provider:model" string.
th0rz05 added a commit that referenced this pull request Jun 9, 2026
…cs (#22)

Bumps the structured-output default model to gemini-3.1-pro-preview and
adds .gitignore rules for .DS_Store and the local-only docs/claude_docs/
scratch directory so internal debug notes are never committed.
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.

2 participants