Skip to content

fix(coding-agents): surface missing sqlite3 instead of silently disabling Devin retain - #3127

Closed
r266-tech wants to merge 2 commits into
vectorize-io:mainfrom
r266-tech:fix/devin-retain-sqlite-errors
Closed

fix(coding-agents): surface missing sqlite3 instead of silently disabling Devin retain#3127
r266-tech wants to merge 2 commits into
vectorize-io:mainfrom
r266-tech:fix/devin-retain-sqlite-errors

Conversation

@r266-tech

@r266-tech r266-tech commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • distinguish a successful empty Devin transcript query from sqlite execution, schema, and parse failures
  • keep Stop hooks fail-open while logging and recording retain_transcript_failed diagnostics
  • probe sqlite3's required JSON query mode during Devin installation and warn without blocking hook setup

The sqlite CLI bridge remains dependency-free for Node; users now get an actionable signal when its external prerequisite cannot read transcripts. Thanks @ichoosetoaccept for isolating the failure mode in #3125.

Testing

  • npm test (267 tests)
  • npx tsc --noEmit
  • npm run build
  • Prettier check on all changed files

CI note

Current main contains #3109's canonical OpenAPI update but not its generated skills/hindsight-docs/references/openapi.json mirror. The required generated-file check reproduced that base drift on this PR, so a separate commit synchronizes the mirror byte-for-byte with hindsight-docs/static/openapi.json; no API source or schema is changed here.

Fixes #3125

@nicoloboschi

Copy link
Copy Markdown
Collaborator

see #3175

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.

coding-agents: Devin CLI retain silently no-ops when the sqlite3 binary is absent

2 participants