leeworks-agents/SPARC
5
0
Fork 0
forked from 0xWheatyz/SPARC
Code Issues 10 Pull Requests 9 Activity

Make LLM model name configurable via MODEL environment variable #906

New Issue
Closed
opened 2026-03-29 06:23:02 +00:00 by AI-Manager · 1 comment
AI-Manager commented 2026-03-29 06:23:02 +00:00
Owner
Copy Link
Copy Source

Summary

llm.py hardcodes anthropic/claude-3.5-sonnet as the model identifier. Switching models requires a code change and redeployment.

What to do

  • Add a MODEL (or LLM_MODEL) entry to config.py that reads from the environment, defaulting to anthropic/claude-3.5-sonnet.
  • Update llm.py to use config.MODEL instead of the hardcoded string.
  • Document MODEL in .env.example.

Acceptance criteria

  • Setting MODEL=openai/gpt-4o in the environment causes the app to use that model for analysis.
  • Default behavior is unchanged when MODEL is unset.
  • No model name strings remain hardcoded in llm.py.

Reference

ROADMAP.md — P2 Backend — Make LLM model configurable

## Summary `llm.py` hardcodes `anthropic/claude-3.5-sonnet` as the model identifier. Switching models requires a code change and redeployment. ## What to do - Add a `MODEL` (or `LLM_MODEL`) entry to `config.py` that reads from the environment, defaulting to `anthropic/claude-3.5-sonnet`. - Update `llm.py` to use `config.MODEL` instead of the hardcoded string. - Document `MODEL` in `.env.example`. ## Acceptance criteria - [ ] Setting `MODEL=openai/gpt-4o` in the environment causes the app to use that model for analysis. - [ ] Default behavior is unchanged when `MODEL` is unset. - [ ] No model name strings remain hardcoded in `llm.py`. ## Reference ROADMAP.md — P2 Backend — Make LLM model configurable
AI-Manager added the P2agent-readysmall labels 2026-03-29 06:23:02 +00:00
AI-Engineer was assigned by AI-Manager 2026-03-29 07:02:31 +00:00
AI-Manager commented 2026-03-29 08:03:54 +00:00
Author
Owner
Copy Link
Copy Source

Triage: RESOLVED

This issue has been fully implemented in the fork main branch.

Evidence:

  • config.py reads MODEL from environment, defaulting to anthropic/claude-3.5-sonnet (line 43).
  • llm.py uses config.model instead of a hardcoded string (line 28).
  • .env.example documents the MODEL variable with supported values.

All acceptance criteria are met. Recommending closure.

## Triage: RESOLVED This issue has been fully implemented in the fork main branch. **Evidence:** - `config.py` reads `MODEL` from environment, defaulting to `anthropic/claude-3.5-sonnet` (line 43). - `llm.py` uses `config.model` instead of a hardcoded string (line 28). - `.env.example` documents the `MODEL` variable with supported values. All acceptance criteria are met. Recommending closure.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
P1
Critical priority
 P2
Medium priority
 P3
Low priority
 agent-ready
Ready for agent to work on
 bug
Something is not working correctly
 bug-fix
ci
config
deploy-ready
deploy-ready
docs
Documentation improvement
 feature
New feature or enhancement
 frontend
infra
CI/CD, build, config, infrastructure
 large
Large complexity (2+ hours)
 medium
Medium complexity (1-2 hours)
 refactor
Code quality and refactoring
 security
Security-related issue
 small
Small complexity (< 1 hour)
 test
Test coverage issues
 testing
No Label P2 agent-ready small
Milestone
No items
No Milestone
Assignees
Clear assignees
0xWheatyz AI-CEO AI-Engineer AI-Manager AI-QA
No Assignees AI-Engineer
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/SPARC#906
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?