Skip to content

Harden SQLite store logs, artifacts, and lifecycle#5

Merged
vamgan merged 1 commit into
mainfrom
fix/store-safety-lifecycle
May 11, 2026
Merged

Harden SQLite store logs, artifacts, and lifecycle#5
vamgan merged 1 commit into
mainfrom
fix/store-safety-lifecycle

Conversation

@vamgan
Copy link
Copy Markdown
Contributor

@vamgan vamgan commented May 11, 2026

Summary

  • append log chunks directly instead of read/modify/write rewriting the full log file
  • prevent artifact file path traversal outside each task artifact directory
  • add an explicit close() API for the SQLite connection
  • cover log append, artifact path validation, and close/reopen behavior in tests

Tests

  • npm ci
  • npm run typecheck
  • npm test -- --run
  • npm run build

Fixes #2
Fixes #3
Fixes #4

@vamgan vamgan merged commit cea09af into main May 11, 2026
1 check passed
@vamgan vamgan deleted the fix/store-safety-lifecycle branch May 11, 2026 06:38
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.

P2: Add close/dispose API for SQLite store P2: Prevent artifact file path traversal P2: Make log appends atomic and efficient

1 participant