Skip to content

[Refactor] Remove git from artifact persistence#147

Merged
samzong merged 2 commits intomainfrom
refactor/remove-git
Mar 23, 2026
Merged

[Refactor] Remove git from artifact persistence#147
samzong merged 2 commits intomainfrom
refactor/remove-git

Conversation

@samzong
Copy link
Copy Markdown
Collaborator

@samzong samzong commented Mar 23, 2026

What's changed?

  • Delete artifact/git.ts (commitArtifact, commitArtifactBatch, promise queue)
  • Remove simple-git dependency
  • Remove gitSha field from Artifact type, Drizzle schema, CREATE TABLE SQL, search, save, stores
  • Remove git init + .gitignore creation from workspace setup
  • Remove git commit calls from artifact handlers and auto-extract
  • Remove empty Git tab from RightPanel
  • Update all architecture docs, project guides, and website copy to reflect two-layer persistence (filesystem + SQLite)

Why

  • Artifact filenames include UUIDs — files are never modified after creation, making git diffs and version history meaningless
  • gitSha was stored in DB but never consumed by any UI or query
  • SQLite + filesystem already handle persistence, search, and traceability
  • Removes simple-git dependency, I/O overhead, and serialization complexity

@github-actions
Copy link
Copy Markdown
Contributor

Hi @samzong,
Thanks for your pull request!
If the PR is ready, use the /auto-cc command to assign Reviewer to Review.
We will review it shortly.

Details

Instructions for interacting with me using comments are available here.
If you have questions or suggestions related to my behavior, please file an issue against the gh-ci-bot repository.

@samzong samzong merged commit 0a88c0f into main Mar 23, 2026
8 checks passed
@samzong samzong deleted the refactor/remove-git branch March 24, 2026 06:20
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.

1 participant