📦 new: add devin cli support and model selection - #10
Open
warengonzaga wants to merge 4 commits into
Open
warengonzaga wants to merge 4 commits into
warengonzaga wants to merge 4 commits into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
warengonzaga
added
enhancement
🔧 Container Build Complete - PR BuildBuild Status: ✅ Success 📦 Pull ImageDocker Hub: docker pull wgtechlabs/glasses:pr-6818050GHCR: docker pull ghcr.io/wgtechlabs/glasses:pr-6818050📋 Build Details
🏷️ Image Tags• 🔍 Testing Your Changes
🚀 Quick Start# Pull and run the container
Docker Hub: docker pull wgtechlabs/glasses:pr-6818050
docker run <your-options> <image>🔒 Security Scan Results📋 Pre-Build Security Checks✅ Source Code Scan: 0 vulnerabilities found 🐳 Container Image Vulnerabilities
📊 Detailed Security ReportsView detailed vulnerability reports in the GitHub Security tab. 🤖 Powered by Container Build Flow Action v7df0af8c4395db7df1927ef215445cd546e9e3f2 |
There was a problem hiding this comment.
Pull request overview
This PR adds Devin CLI as an additional main-session backend alongside Copilot, with Telegram commands to pin the active agent/repository and optionally select a model per conversation, persisting these settings in Postgres and propagating them through the scheduler/runner pipeline.
Changes:
- Add Telegram
/newand/modelcontrols to configure per-chat agent/repository/model. - Extend conversation persistence (DB + types) to include
agent,repository, and nullablemodel, and pass these through to runner inputs. - Implement a Devin main-session execution path in the runner (with default model
swe-1.7) and optionally inject Devin credentials into sandboxes.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/telegram.test.ts | Updates Telegram control-chat tests for /new and /model. |
| src/types.ts | Introduces AgentName and extends Conversation with agent/repository/model. |
| src/scheduler.ts | Passes agent/model/repository into runner inputs; adjusts sandbox creation token behavior. |
| src/sandbox.ts | Makes main sandbox token optional and passes optional DEVIN_CREDENTIALS_BASE64 into sandbox env. |
| src/runner.ts | Adds runner input validation for agent/model and a Devin main execution path (devin -p). |
| src/runner-protocol.ts | Extends RunnerInput protocol with agent and model. |
| src/db.ts | Adds model column and adds configureMainConversation for migration-safe per-chat config updates. |
| src/channels/telegram.ts | Implements /new and /model handlers and surfaces agent/model/repository in /status. |
| README.md | Documents new Telegram commands and Devin support/environment variables. |
| docs/ARCHITECTURE.md | Updates architecture docs to describe Copilot vs Devin main-session behavior. |
| .env.example | Adds DEVIN_CREDENTIALS_BASE64 guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- /new and /model now destroy the previously running main sandbox instead of only nulling its DB pointer, preventing orphaned idle sandboxes until timeout - ensureDevinReady surfaces a clear auth-failure message on stderr instead of redirecting devin auth status to /dev/null - README: describe the main session as agent-agnostic (Copilot by default) and drop the stale '/new is compatibility-only' line Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
The DEVIN_CREDENTIALS_BASE64 value in tests/config.test.ts is a fake base64 fixture ('devin-creds'), not a real secret, but Gitleaks' generic-api-key rule flags the credential-like assignment and fails the CI gate.
- Add .gitleaksignore with the finding fingerprint to suppress the original introduction in the branch's commit-range scan.
- Add an inline gitleaks:allow on the fixture line so future/squashed history and full-tree scans stay clean.
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.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
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
epository, and nullable model (with migration-safe DB updates)
Validation
Need help on this PR? Tag
/codesmithwith what you need. Autofix is enabled.