mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-10-06 21:05:21 +02:00
## Thinking Path > - Paperclip shows company activity in the dashboard. > - The dashboard reads company, task, approval, and cost data. > - A pooled database connection can close during one of these reads. > - The driver must reject ambiguous statements because writes may have committed. > - These four dashboard queries are known to be read-only. > - This change retries only the failed read and preserves completed work. ## Linked Issues or Issue Description Refs #14773, which correctly removed automatic replay of ambiguous database statements. Searched existing database and dashboard PRs. Related #8780 changes pool recycling and logging; #13925 adds dashboard consistency coverage. Neither provides these per-query retries. **What happened?** A dashboard request returns a server error when its company lookup, task count, approval count, or monthly spend query loses its database connection. Drizzle wraps the driver's connection error in `cause`. **Expected behavior** A transient connection failure gets a bounded retry of the specific read. Completed reads and budget processing are not replayed. Persistent outages and non-connection errors still fail the request. **Steps to reproduce** Inject a typed `CONNECTION_CLOSED` error into one of these reads. Then allow the next query to succeed. Before this change, the dashboard request fails immediately. ## What Changed - Apply independent retries to the company lookup, task counts, pending approval count, and monthly spend query. Each callback rebuilds its own query with the same company scope. - Extract the existing authentication retry helper as `retryIdempotentDatabaseOperation`. Preserve authentication behavior and its existing exports. - Retain the existing limit of three total attempts with 50 ms and 100 ms pauses. Match typed connection codes through the error cause chain. - Test later-read failures, unchanged query parameters, retry exhaustion, missing companies, and errors that must not retry. Document the boundary. ## Verification - Final focused dashboard, authentication, and real database wire suites: 38 tests passed. Six initial recovery regressions failed before the implementation. - `pnpm -r typecheck`: passed on the final source. - Independent review: no actionable findings. The reviewer separately passed all 38 focused tests and checked the code allowlist, attempt bounds, pauses, and final error identity. - `pnpm test:run`: the general-server group completed with 14,738 tests passed, 13 failed, and 87 skipped. All 13 failures match the previously reproduced clean-base macOS skill-cache failures. The two test files and their implementations are unchanged from that baseline. The runner exited after this group, so the remaining local workspace and serialized groups did not run. All corresponding Linux CI groups passed on this commit. - `pnpm build`: passed on the final source. - Full CI: 53 successful checks and 2 skips on `6a113529c0`. Greptile: 5/5 on that commit, with no review threads or remaining findings. - Merge compatibility with master `f2e0f19630`, including #14866: no conflicts. The five reviewed files are unchanged in the resulting merge tree. ## Risks A persistent outage adds at most two retries per covered query. Each connection attempt retains the configured driver timeout. The change does not repair the underlying network or database failure. Agent counts, run-activity queries, and the budget workflow stay outside these retry boundaries. General database statements and disconnected transactions are not replayed. There is no schema or authorization change. ## Model Used OpenAI GPT-6 through Codex, with reasoning, repository inspection, code editing, and test execution. The runtime does not expose a more specific serving model identifier or context-window size. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work - [x] I have searched GitHub for duplicate or related PRs and linked them above - [x] I have either (a) linked existing issues with `Fixes: #` / `Closes #` / `Refs #` OR (b) described the issue in-PR following the relevant issue template - [x] I have not referenced internal/instance-local Paperclip issues or links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip` URLs) - [x] My branch name describes the change and contains no internal Paperclip ticket id or instance-derived details - [x] I have run tests locally and they pass (38 focused tests; the full local run has the baseline limitation documented above) - [x] I have added or updated tests where applicable - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [x] All Paperclip CI gates are green - [x] Greptile is 5/5 with no open P2s, recommendations, or follow-ups - [x] I will address all Greptile and reviewer comments before requesting merge Co-authored-by: Paperclip <noreply@paperclip.ing>
494 lines
27 KiB
Markdown
494 lines
27 KiB
Markdown
# Database
|
|
|
|
Paperclip uses PostgreSQL via [Drizzle ORM](https://orm.drizzle.team/). There are three ways to run the database, from simplest to most production-ready.
|
|
|
|
## 1. Embedded PostgreSQL — zero config
|
|
|
|
If you don't set `DATABASE_URL`, the server automatically starts an embedded PostgreSQL instance and manages a local data directory.
|
|
|
|
```sh
|
|
pnpm dev
|
|
```
|
|
|
|
That's it. On first start the server:
|
|
|
|
1. Creates a `~/.paperclip/instances/default/db/` directory for storage
|
|
2. Ensures the `paperclip` database exists
|
|
3. Runs migrations automatically for empty databases
|
|
4. Starts serving requests
|
|
|
|
Data persists across restarts in `~/.paperclip/instances/default/db/`. To reset local dev data, delete that directory.
|
|
|
|
If you need to apply pending migrations manually, run:
|
|
|
|
```sh
|
|
pnpm db:migrate
|
|
```
|
|
|
|
When `DATABASE_URL` is unset, this command targets the current embedded PostgreSQL instance for your active Paperclip config/instance.
|
|
|
|
Issue reference mentions follow the normal migration path: the schema migration creates the tracking table, but it does not backfill historical issue titles, descriptions, comments, or documents automatically.
|
|
|
|
To backfill existing content manually after migrating, run:
|
|
|
|
```sh
|
|
pnpm issue-references:backfill
|
|
# optional: limit to one company
|
|
pnpm issue-references:backfill -- --company <company-id>
|
|
```
|
|
|
|
Future issue, comment, and document writes sync references automatically without running the backfill command.
|
|
|
|
This mode is ideal for local development and one-command installs.
|
|
|
|
Docker note: the Docker quickstart image also uses embedded PostgreSQL by default. Persist `/paperclip` to keep DB state across container restarts (see `doc/DOCKER.md`).
|
|
|
|
## 2. Local PostgreSQL (Docker)
|
|
|
|
For a full PostgreSQL server locally, use the included Docker Compose setup:
|
|
|
|
```sh
|
|
docker compose up -d
|
|
```
|
|
|
|
This starts PostgreSQL 17 on `localhost:5432`. Then set the connection string:
|
|
|
|
```sh
|
|
cp .env.example .env
|
|
# .env already contains:
|
|
# DATABASE_URL=postgres://paperclip:paperclip@localhost:5432/paperclip
|
|
```
|
|
|
|
Run migrations:
|
|
|
|
```sh
|
|
DATABASE_URL=postgres://paperclip:paperclip@localhost:5432/paperclip \
|
|
pnpm db:migrate
|
|
```
|
|
|
|
Start the server:
|
|
|
|
```sh
|
|
pnpm dev
|
|
```
|
|
|
|
## 3. Hosted PostgreSQL (Supabase)
|
|
|
|
For production, use a hosted PostgreSQL provider. [Supabase](https://supabase.com/) is a good option with a free tier.
|
|
|
|
### Setup
|
|
|
|
1. Create a project at [database.new](https://database.new)
|
|
2. Go to **Project Settings > Database > Connection string**
|
|
3. Copy the URI and replace the password placeholder with your database password
|
|
|
|
### Connection string
|
|
|
|
Supabase offers two connection modes:
|
|
|
|
**Direct connection** (port 5432) — use for migrations and one-off scripts:
|
|
|
|
```
|
|
postgres://postgres.[PROJECT-REF]:[PASSWORD]@aws-0-[REGION].pooler.supabase.com:5432/postgres
|
|
```
|
|
|
|
**Connection pooling via Supavisor** (port 6543) — use for the application:
|
|
|
|
```
|
|
postgres://postgres.[PROJECT-REF]:[PASSWORD]@aws-0-[REGION].pooler.supabase.com:6543/postgres
|
|
```
|
|
|
|
### Configure
|
|
|
|
For the application runtime, use a direct PostgreSQL connection unless the database client has explicit prepared-statement configuration for your pooling mode:
|
|
|
|
```sh
|
|
DATABASE_URL=postgres://postgres.[PROJECT-REF]:[PASSWORD]@aws-0-[REGION].pooler.supabase.com:5432/postgres
|
|
```
|
|
|
|
If you later run the app with a pooled runtime URL, set `DATABASE_MIGRATION_URL` to the direct connection URL. Paperclip uses it for startup schema checks/migrations and plugin namespace migrations, while the app continues to use `DATABASE_URL` for runtime queries:
|
|
|
|
```sh
|
|
DATABASE_URL=postgres://postgres.[PROJECT-REF]:[PASSWORD]@aws-0-[REGION].pooler.supabase.com:6543/postgres
|
|
DATABASE_MIGRATION_URL=postgres://postgres.[PROJECT-REF]:[PASSWORD]@aws-0-[REGION].pooler.supabase.com:5432/postgres
|
|
```
|
|
|
|
If your hosted database requires transaction-pooling-only connections (pgbouncer transaction mode, Supavisor port 6543, Neon `-pooler` endpoints), set `DATABASE_PREPARED_STATEMENTS=false` so the client does not rely on session-scoped prepared statements, and keep `DATABASE_MIGRATION_URL` on a direct connection. Do not edit database client source files as part of deployment setup.
|
|
|
|
### Client tuning (optional)
|
|
|
|
All of these are optional; when unset, the driver defaults apply and behavior is unchanged — typical self-hosted setups need none of them:
|
|
|
|
```sh
|
|
DATABASE_PREPARED_STATEMENTS=false # required for transaction-mode poolers; default: enabled
|
|
DATABASE_POOL_MAX=25 # connection pool size; default: 10
|
|
DATABASE_IDLE_TIMEOUT_SECONDS=60 # close idle pooled connections; default: 60 (0 = keep open)
|
|
DATABASE_CONNECT_TIMEOUT_SECONDS=10 # default: 30
|
|
DATABASE_MAX_LIFETIME_SECONDS=1800 # recycle a pooled connection after this long; default: 30-60 min (random)
|
|
DATABASE_APPLICATION_NAME=paperclip # application_name in pg_stat_activity; default: paperclip
|
|
```
|
|
|
|
### Push the schema
|
|
|
|
```sh
|
|
# Use the direct connection (port 5432) for schema changes
|
|
DATABASE_URL=postgres://postgres.[PROJECT-REF]:[PASSWORD]@...5432/postgres \
|
|
pnpm db:migrate
|
|
```
|
|
|
|
### Free tier limits
|
|
|
|
- 500 MB database storage
|
|
- 200 concurrent connections
|
|
- Projects pause after 1 week of inactivity
|
|
|
|
See [Supabase pricing](https://supabase.com/pricing) for current details.
|
|
|
|
## Connection loss and retries
|
|
|
|
The database client does not replay arbitrary statements after a disconnect.
|
|
PostgreSQL may have committed a statement before the connection loses its
|
|
response. The postgres.js message `write CONNECTION_CLOSED` does not prove
|
|
that the statement was never sent: the driver also uses it when an in-flight
|
|
query loses its connection. SQL text cannot establish replay safety either;
|
|
a `SELECT` can call a function with side effects.
|
|
|
|
The affected operation fails and a new operation can reconnect through the
|
|
pool. Callers may retry only when the complete operation is idempotent or has
|
|
a durable receipt that prevents duplicate effects. Some transient statement
|
|
failures therefore reach the caller instead of being retried automatically.
|
|
|
|
When a database connection closes, its transaction fails. Paperclip does not
|
|
replay that transaction. New requests can use a fresh connection from the pool.
|
|
Queries from the failed transaction must keep failing, even after the pool
|
|
reconnects.
|
|
|
|
Source builds carry `patches/postgres@3.4.9.patch` for this behavior. It rejects
|
|
queued and later queries from a disconnected transaction or reserved connection,
|
|
and prevents a released, closed connection from returning to the open pool.
|
|
The patch covers both ESM and CommonJS. The regression suite terminates real
|
|
PostgreSQL backends and checks rejection, pool recovery, and transaction isolation.
|
|
Remove the patch when an upstream release passes these tests. Installs of the
|
|
unmodified `postgres` package outside this workspace do not include the patch.
|
|
|
|
Trusted-header actor synchronization retries transient connection failures,
|
|
including `CONNECT_TIMEOUT`, at most twice. This retry applies only to the
|
|
idempotent actor synchronization operations, not arbitrary transactions. A
|
|
persistent outage still fails the request after the bounded retries; each
|
|
connection attempt remains subject to the configured database connect timeout.
|
|
|
|
The dashboard's company lookup, task counts, pending approval count, and
|
|
monthly spend each retry these connection errors at most twice. Each callback
|
|
is read-only and rebuilds its query for each attempt. A failed read
|
|
does not replay completed reads or the budget workflow. Missing companies,
|
|
authentication errors, and other database errors propagate without retry.
|
|
This does not enable general SQL replay.
|
|
|
|
## Execution identity row locks
|
|
|
|
Identity initialization, credential acquisition, and steering reconciliation lock
|
|
the task before its run. These operations use `FOR NO KEY UPDATE`: they change
|
|
identity state, not parent keys. The lock still serializes identity writers and
|
|
blocks concurrent task or run updates. It allows audit inserts to retain their
|
|
foreign-key `KEY SHARE` locks without waiting on identity acquisition. The audit
|
|
foreign keys and their deletion behavior remain enforced.
|
|
|
|
## Switching between modes
|
|
|
|
The database mode is controlled by `DATABASE_URL`:
|
|
|
|
| `DATABASE_URL` | Mode |
|
|
|---|---|
|
|
| Not set | Embedded PostgreSQL (`~/.paperclip/instances/default/db/`) |
|
|
| `postgres://...localhost...` | Local Docker PostgreSQL |
|
|
| `postgres://...supabase.com...` | Hosted Supabase |
|
|
|
|
Your Drizzle schema (`packages/db/src/schema/`) stays the same regardless of mode.
|
|
|
|
## Migration authoring checklist
|
|
|
|
The 0126 issue comment attribution backfill showed the failure mode this checklist is meant to prevent: each batch looked for the next rows with an unindexed predicate, so PostgreSQL repeatedly scanned the same table and the migration became O(n²) as the table grew.
|
|
|
|
When authoring migrations or one-time backfills:
|
|
|
|
- Create the supporting index for the batch predicate before the backfill loop runs.
|
|
- Bound batches by an indexed key, such as an id range or keyset pagination cursor. Do not use `OFFSET` pagination or a query shape that re-scans already-visited rows each batch.
|
|
- Avoid unbounded full-table `UPDATE` or `DELETE` statements. Add a selective predicate and process rows in bounded batches when table size can be large.
|
|
- Use `CREATE INDEX CONCURRENTLY` for large existing tables when the migration can run outside a transaction and must avoid long write locks.
|
|
- Split schema changes, index creation, and data backfill into separate phases so each step has clear locking and rollback behavior.
|
|
- Treat the `check:migrations` CI gate as the enforcement backstop for these rules. If it flags a migration, rewrite the migration or add a suppression comment with the indexed predicate, batch bound, and reason the remaining scan is safe.
|
|
|
|
## Migration snapshots
|
|
|
|
`drizzle-kit generate` diffs `packages/db/src/schema/` against the newest snapshot in `packages/db/src/migrations/meta/`. That snapshot must describe the schema that every migration produces when they run in order. A snapshot that drifts from the schema makes the *next* migration wrong, because `generate` folds the drift into it. The drift can add a column that an earlier migration already created, which makes that migration fail on a fresh database. It can also drop a column that the schema still uses.
|
|
|
|
- Create every migration with `pnpm --filter @paperclipai/db generate`. Do not hand-write a snapshot.
|
|
- Do not hand-edit a snapshot to resolve a merge conflict. Renumber your migration and run `generate` again, as `packages/db/.gitattributes` describes.
|
|
- The repo keeps only the newest 5 snapshots. `generate` runs `prune:snapshots` afterwards to delete older ones. Drizzle only reads the newest snapshot, and each snapshot is a full copy of the schema (over 1 MB each). Older snapshots are still in git history.
|
|
- `packages/db/src/migration-snapshot-drift.test.ts` is the enforcement backstop. It repeats the diff that `generate` performs and fails when the newest snapshot no longer matches `packages/db/src/schema/`.
|
|
|
|
## Cloud runtime identity singleton
|
|
|
|
The private `instance_settings` row whose singleton key is
|
|
`cloud-runtime-identity/v1` records the immutable Cloud stack id, warm-pool
|
|
claim id, previous pool origin, canonical origin, and stack slug accepted from
|
|
Cloud's signed pre-activation assertion. It is separate from the normal
|
|
`default` settings row and never appears in the settings API. This is
|
|
intentionally instance-scoped rather than company-scoped: an instance has one
|
|
public identity, and the existing unique singleton-key index makes concurrent
|
|
or later attempts to replace it fail closed. The server loads the row before
|
|
constructing URL-dependent runtime services on every boot.
|
|
|
|
## Resource membership tables
|
|
|
|
Paperclip stores current-user sidebar membership state in:
|
|
|
|
- `project_memberships`
|
|
- `agent_memberships`
|
|
|
|
These rows are company-scoped and user-scoped. A missing row means the user is joined, so existing users keep seeing projects and agents in the sidebar until they explicitly leave them. Rows only control sidebar visibility; they do not affect project/agent detail access, all-pages, selectors, assignment flows, or existing company permissions.
|
|
|
|
Both tables use a unique key on `(company_id, user_id, resource_id)` and keep `state` as `joined` or `left`. Join/leave mutations are idempotent board-user `/me` operations and write activity entries when the effective state changes.
|
|
|
|
## Decision training snapshot retention
|
|
|
|
`decision_training_examples` stores a point-in-time copy of an issue, its comments, relevant runs, and the selected decision. Each row carries the `scrub_deleted_comments_v1` retention policy marker, and JSONL exports include that marker alongside the snapshot.
|
|
|
|
- Deleting a captured source comment transactionally replaces that comment in every affected snapshot with a content-free redaction tombstone. The original body, presentation, and metadata are not retained in the training record.
|
|
- Deleting an issue deletes its decision-training examples through the `issue_id` foreign-key cascade.
|
|
- Deleting a training example deletes only that example and does not mutate the source issue.
|
|
|
|
This policy makes training exports self-describing while keeping the decision record usable after a comment deletion without retaining content the author removed.
|
|
|
|
## Decision queues and triage provenance
|
|
|
|
The decisions desk stores queue membership, decide-by/snooze state, and retention state in `decision_queues`, `decision_queue_items`, `decision_triage`, and `decision_retention`. These sidecars use the stable attention identity `(source_kind, source_id)` so all attention source kinds can participate without copying source titles, bodies, projects, or other visibility-sensitive data.
|
|
|
|
`decision_triage_events` is append-only history for queue and triage changes. Current rows and history both carry server-derived user/agent, heartbeat run, API-key, and responsible-user attribution where applicable. Queue reads must resolve and authorize their source rows at read time; a sidecar row is never a visibility grant.
|
|
|
|
Triage writes serialize on the company and attention-source identity so concurrent partial updates preserve both fields and produce monotonic history versions.
|
|
|
|
`decision_retention` tracks the last observed source `activityAt`, Keep, reversible archive provenance, and monotonic source/archive versions. `decision_archive_notification_outbox` has a unique key over company, source identity, archive version, and immutable origin agent so repeated sweeps cannot enqueue duplicate notifications; delivery claims are retryable and coalesced per agent.
|
|
|
|
## Native runner persistence
|
|
|
|
Native runner state is additive to the existing heartbeat tables. Every existing
|
|
`heartbeat_runs` row defaults to `runtime_mode = 'legacy'`; adding these columns
|
|
does not select the native runtime or start a runner process. Native execution can
|
|
record its resolved runtime profile, provider session, driver, completion
|
|
contract, durable event cursor, and finalization phase on the run when a later
|
|
rollout explicitly selects it.
|
|
|
|
`completion_contracts`, `native_run_results`, `native_run_finalizations`,
|
|
`work_assessments`, `status_decisions`, and `status_decision_effects` form the
|
|
append-oriented evidence and status-decision chain. Unique fingerprints,
|
|
versions, ordinals, and idempotency keys make retries deterministic. Composite
|
|
foreign keys bind every contract, result, assessment, decision, effect, and
|
|
finalization to one company, issue, and run. The database rejects mixed-owner
|
|
evidence even when every referenced ID exists. Native source identities on
|
|
`heartbeat_run_events` are nullable so legacy events remain readable without
|
|
rewriting historical rows. Per-run native source identifiers are unique, while
|
|
the existing legacy sequence behavior remains unchanged. The hidden native
|
|
coordinator serializes on its bound `heartbeat_runs` row, allocates
|
|
`next_event_seq`, and commits a validated PRP event before the transport sends
|
|
its cumulative ACK. Byte-equivalent source retries return the existing cursor;
|
|
gaps and conflicting replays fail closed. Accepted structured results enter the
|
|
finalization ledger, whose retry time and owner lease are checked under a row
|
|
lock. None of these writes selects a runtime or changes a legacy run's execution
|
|
path.
|
|
|
|
Durable agent session goals are an additive projection on
|
|
`agent_task_sessions`, distinct from the business-goal hierarchy. The row stores
|
|
the negotiated goal capability, normalized snapshot and status, desired state,
|
|
provider source cursor, monotonic projection revision, and observation time.
|
|
`agent_session_goal_actions` is the control outbox: `(session_id, request_id)`
|
|
is unique, so retries return the original accepted action. Provider source
|
|
ordering fences duplicate and stale updates, and a cleared projection retains
|
|
its revision/cursor tombstone so an older provider event cannot resurrect it.
|
|
|
|
Issue `status_version` advances only when `status` changes. The JavaScript backup
|
|
path includes user-defined functions and triggers so a restored database keeps
|
|
that invariant. Removing or disabling a future native rollout flag must not
|
|
delete these records; persisted experimental runs remain available for recovery
|
|
and inspection.
|
|
|
|
`native_run_finalizations` also stores restart ownership and recovery state.
|
|
The controller owner is a server boot id, PID, operating-system process-start
|
|
timestamp, and monotonically increasing controller generation. Recovery writes
|
|
its correlated request id, current state, and a bounded JSON history. A
|
|
successor can take the lease immediately only when coordinated handoff or PID
|
|
and process-start evidence proves the prior controller is gone, or when the
|
|
lease expires. Recovery generation changes do not increment the independent
|
|
provider-attempt counter.
|
|
|
|
## Chat communication snapshots
|
|
|
|
Chat communication guidance uses two additive columns: endpoint
|
|
`communication_instructions` defaults to empty, and conversation
|
|
`communication_guidance` holds the immutable initial task snapshot. Existing
|
|
conversations retain a null snapshot; there is no backfill that changes an
|
|
ongoing conversation. New Slack tasks receive built-in guidance even when the
|
|
endpoint has no additional instructions.
|
|
|
|
## Telegram private draft identities
|
|
|
|
`chat_telegram_draft_ids` is a content-free, instance-wide PostgreSQL sequence,
|
|
not a company-owned record. Telegram's native Stop callback carries a draft ID
|
|
but no actor or Paperclip generation. IDs therefore must not be recycled when
|
|
a transaction rolls back or an endpoint/company is deleted and its bot is
|
|
connected again. The sequence allocates positive 31-bit IDs without cycling;
|
|
exhaustion refuses new draft allocation rather than wrapping or falling back to
|
|
random IDs. Never reset it as part of chat cleanup.
|
|
|
|
The matching `chat_actions` entry remains company/endpoint-scoped and binds the
|
|
draft to its exact conversation, publication attempt, runtime, credential and
|
|
approved text. Stop can suppress that private draft's final publication; it
|
|
cannot cancel a task or model run. Logical backups preserve the sequence, but
|
|
restoring an older database may roll back its high-water mark: disaster recovery
|
|
must not assume stale provider Stop events are safe to reuse. That restore
|
|
boundary is not qualified by the rollback/concurrency regression.
|
|
|
|
## Attachment upload provenance
|
|
|
|
`issue_attachments.originating_run_id` records server-derived run attribution at
|
|
upload time. It is not writable through attachment or work-product update APIs.
|
|
Legacy attachments and uploads without a registered run keep a null value; the
|
|
migration deliberately does not infer attribution from mutable work products.
|
|
Deleting the originating run clears the reference and fails closed for automatic
|
|
chat handoff. An agent's external file selection must match the attachment's
|
|
company, task, agent, and originating run. Editing or recreating a work-product
|
|
record cannot reassign that authority to a later run.
|
|
|
|
## Question-response delivery receipts
|
|
|
|
`issue_question_response_deliveries` is the retry-safe, content-free outbox for
|
|
answered `ask_user_questions` interactions. Its unique interaction and correlation
|
|
indexes enforce one causal delivery per response. It records source and target
|
|
run/turn ids, payload digest, attempt/acknowledgement state, and one of `steered`,
|
|
`coalesced`, or `wake_fallback`; answer content remains only in
|
|
`issue_thread_interactions.result`. Deleting the interaction cascades its receipt,
|
|
while deleting a referenced run clears that run pointer without deleting history.
|
|
|
|
## Plugin database namespaces
|
|
|
|
The plugin runtime tracks plugin-owned database namespaces and migrations in `plugin_database_namespaces` and `plugin_migrations`. Hosted deployments that separate runtime and migration connections should set `DATABASE_MIGRATION_URL`; plugin namespace migration work uses the migration connection when present.
|
|
|
|
## Backups
|
|
|
|
Paperclip supports automatic and manual logical database backups. These dumps include
|
|
non-system database schemas such as `public`, the Drizzle migration journal, and
|
|
plugin-owned database schemas. See `doc/DEVELOPING.md` for the current
|
|
`paperclipai db:backup` / `pnpm db:backup` commands and backup retention
|
|
configuration.
|
|
|
|
Database backups do not include non-database instance files such as local-disk
|
|
uploads, workspace files, or the local encrypted secrets master key. Back those paths
|
|
up separately when you need full instance disaster recovery.
|
|
|
|
## Secret storage
|
|
|
|
Paperclip stores secret metadata and versions in:
|
|
|
|
- `user_secret_definitions`
|
|
- `user_secret_declarations`
|
|
- `company_secrets`
|
|
- `company_secret_versions`
|
|
- `company_secret_bindings`
|
|
- `secret_access_events`
|
|
|
|
Company secrets use `company_secrets.scope = 'company'` and are bound directly
|
|
through `company_secret_bindings`. User-specific secrets reuse the same provider
|
|
and version storage, but each value is a `company_secrets.scope = 'user'` row
|
|
with `owner_user_id` and `user_secret_definition_id` set. Definitions describe
|
|
the reusable company-level slot, declarations record where `user_secret_ref`
|
|
bindings are required, and the concrete value is selected later for the
|
|
responsible user.
|
|
|
|
Secret-aware env bindings are supported by agents, projects, and routines. Routine env lives in `routines.env`, is captured in `routine_revisions.snapshot`, and routine dispatches store `routine_runs.routine_revision_id` so runtime secret resolution uses the env snapshot that existed when the run was created. Routine secret refs bind with `target_type = 'routine'`, `target_id = routines.id`, and `config_path` values under `env.*`.
|
|
|
|
For local/default installs, the active provider is `local_encrypted`:
|
|
|
|
- Secret material is encrypted at rest with a local master key.
|
|
- Default key file: `~/.paperclip/instances/default/secrets/master.key` (auto-created if missing).
|
|
- CLI config location: `~/.paperclip/instances/default/config.json` under `secrets.localEncrypted.keyFilePath`.
|
|
- Backup/restore requires both the database metadata and the local master key file; either artifact alone is insufficient.
|
|
- The server best-effort enforces `0600` key file permissions and provider health reports permission warnings.
|
|
- User-scoped values use the same local encrypted provider path. Database
|
|
backups preserve definitions, declarations, owner metadata, version metadata,
|
|
and access events, but restored user-scoped values are decryptable only when
|
|
the matching local master key is restored with the database.
|
|
|
|
Optional overrides:
|
|
|
|
- `PAPERCLIP_SECRETS_MASTER_KEY` (32-byte key as base64, hex, or raw 32-char string)
|
|
- `PAPERCLIP_SECRETS_MASTER_KEY_FILE` (custom key file path)
|
|
|
|
Strict mode to block new inline sensitive env values:
|
|
|
|
```sh
|
|
PAPERCLIP_SECRETS_STRICT_MODE=true
|
|
```
|
|
|
|
You can set strict mode and provider defaults via:
|
|
|
|
```sh
|
|
pnpm paperclipai configure --section secrets
|
|
```
|
|
|
|
Inline secret migration command:
|
|
|
|
```sh
|
|
npx paperclipai secrets migrate-inline-env --company-id <company-id> --apply
|
|
|
|
# direct database maintenance fallback
|
|
pnpm secrets:migrate-inline-env --apply
|
|
```
|
|
|
|
Hosted AWS provider notes live in [SECRETS-AWS-PROVIDER.md](./SECRETS-AWS-PROVIDER.md).
|
|
|
|
### Persistent agent conversations
|
|
|
|
Migration `0274_agent_chat.sql` adds conversation identity/state and session generation/boundary columns to `issues`, plus idempotent client request IDs and processed session-boundary generations to `issue_comments`. The company/agent/user unique index resolves concurrent first writes to one issue. A check constraint preserves the assigned-agent identity and prevents terminal conversation status. Comment request IDs are unique per issue and user. There is no separate chat/message store. Provider sessions continue to use `agent_task_sessions`; `/new` removes only the matching conversation session, and session writers fence stale generations against the issue row.
|
|
|
|
## Legacy controller ownership
|
|
|
|
Legacy run claims atomically record `controller_boot_id`, a database-clock
|
|
`controller_lease_expires_at`, and `execution_stage` before workspace provisioning.
|
|
The lease renews independently of output. A different container must not infer
|
|
controller death from its own process map or numeric PIDs. Expiration grants
|
|
cleanup authority; it does not prove that remote inference has stopped. Recovery
|
|
revokes the previous boot identity with a conditional update. Its own claim also
|
|
expires so another sweep can finish cleanup after a restart. Historical rows keep
|
|
null ownership fields and follow the previous recovery path.
|
|
|
|
## Agent file persistence and legacy revisions
|
|
|
|
Managed agent files are current filesystem contents, using the same persistent
|
|
instance storage as other workspaces. `agent_instruction_revisions` and
|
|
`agent_instruction_heads` are retained as read-only upgrade input. Their heads
|
|
are adopted once into the managed directory; new saves never append revisions.
|
|
`agent_instruction_working_copies` holds per-run baseline hashes, state, and
|
|
capture receipts. New receipts identify `paperclip.agent-files.v1`; historical
|
|
rows retain the instruction-only format. Completed directory runs discard their
|
|
baseline and private copies. See [Persistent agent files](agent-files.md).
|
|
|
|
## Large API response snapshots
|
|
|
|
`assets.byte_size` uses PostgreSQL `bigint` so saved responses and byte ranges can
|
|
exceed 2 GiB. The API and Drizzle mapping continue to expose a JavaScript number;
|
|
response readers validate safe integer offsets. The type-widening migration
|
|
rewrites the asset metadata table and needs an exclusive table lock. File bytes
|
|
remain in local or object storage.
|
|
|
|
### Runner API response reservations
|
|
|
|
`runner_api_response_reservations` holds company-scoped API snapshot reservations.
|
|
Before a capture spills, the server locks company admission and counts stored
|
|
`runner-api` assets plus unattached reservations against a 20 GiB default quota.
|
|
A committed asset replaces its reservation in that total. The asset foreign key
|
|
cascades on deletion, while deleting a run sets `run_id` to null so an orphan
|
|
reservation cannot silently disappear. Failed cleanup or an ambiguous storage
|
|
write requires operator reconciliation before an unattached reservation is
|
|
removed. The table stores no response bodies. See `doc/runner-api-tools.md` for
|
|
limits and the operator override.
|