Skip to content

[dirac] soft-delete-entities#60

Draft
pike00 wants to merge 7 commits into
mainfrom
dirac/soft-delete-entities
Draft

[dirac] soft-delete-entities#60
pike00 wants to merge 7 commits into
mainfrom
dirac/soft-delete-entities

Conversation

@pike00
Copy link
Copy Markdown
Owner

@pike00 pike00 commented May 3, 2026

Auto-generated by the Dirac runner on 2026-05-03T11:26:11Z.

Project: soft-delete-entities
Status before run: active
Outcome: timeout
Commits: 2
Spec: docs/projects/soft-delete-entities/README.md

Generated autonomously with the free-tier OpenRouter model tencent/hy3-preview:free. Review carefully before merging.

Log: .dirac-runner/logs/soft-delete-entities.log

pike00 added 7 commits May 3, 2026 05:36
…, Note

- Add deleted_at field to model classes with index

- Add deleted_at to Public response classes

- Update API endpoints to use soft delete instead of hard delete

- Add restore endpoints (POST /{entity}/{id}/restore)

- Create Alembic migration f6a7b8c9d0e1

- Update deps.py to use filtered session with soft-delete criteria
- Added deleted_at field to Interaction, Reminder, Gift, Debt, LifeEvent, Note models
- Added restore endpoints for all entities (POST /{entity}/{id}/restore)
- Changed delete endpoints to soft delete (set deleted_at) instead of hard delete
- Added SessionLocal and configure_session to db.py
- Fixed current_user parameter issues in route files
- All ruff checks pass
# Conflicts:
#	backend/app/api/routes/contacts.py
#	frontend/src/client/sdk.gen.ts
#	frontend/src/client/types.gen.ts
…ities

# Conflicts:
#	frontend/src/client/schemas.gen.ts
#	frontend/src/client/sdk.gen.ts
#	frontend/src/client/types.gen.ts
@pike00 pike00 force-pushed the dirac/soft-delete-entities branch from 93f3ff3 to 9d80e7e Compare May 7, 2026 03:27
@pike00
Copy link
Copy Markdown
Owner Author

pike00 commented May 9, 2026

🤖 pr-sweep-runner skipped this PR — could not bring it green.

Reason: could not merge origin/main (unresolvable conflicts)

Audit trail: .pr-sweep-runner/logs/, .pr-sweep-runner/patches/, .pr-sweep-runner/replies/

1 similar comment
@pike00
Copy link
Copy Markdown
Owner Author

pike00 commented May 10, 2026

🤖 pr-sweep-runner skipped this PR — could not bring it green.

Reason: could not merge origin/main (unresolvable conflicts)

Audit trail: .pr-sweep-runner/logs/, .pr-sweep-runner/patches/, .pr-sweep-runner/replies/

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