feat: add dry-run workflows

[s0up4200]

Adds dry run toggle and logging for workflow automations.

Closes #1393

Summary by CodeRabbit

• New Features

  • Per-workflow dry-run mode with detailed dry-run activity recording and non-mutating previews.

• UI

  • Dry-run badge, descriptive notes and adjusted dialog titles; dry-run summaries shown across tags, categories, limits, moves, programs, and deletes.
  • Dry run toggle and enable-with/without-dry-run prompt when enabling workflows.

• Import/Export

  • Workflow export/import preserves dry-run setting.

• Database

  • Added dry_run column to automations schema.

• Types

  • Dry-run outcome and related fields surfaced in activity and automation types.

• Tests

  • Tests added for dry-run activity recording.

[coderabbitai]

Walkthrough

Adds per-automation dry-run support end-to-end: DB migration and model fields, API payload, service orchestration to record dry-run activities without mutating state, tests, export/import and type updates, and UI changes to toggle and display dry-run results.

Changes

Cohort / File(s)	Summary
Database & Migration internal/database/migrations/058_add_automation_dry_run.sql, internal/database/db_test.go	Adds dry_run INTEGER NOT NULL DEFAULT 0 column to automations table and updates test schema.
Core Model internal/models/automation.go	Adds exported DryRun bool to Automation and propagates dry_run through ListByInstance/Get/Create/Update SQL paths.
Activity Model internal/models/automation_activity.go	Adds ActivityOutcomeDryRun = "dry-run" to activity outcomes.
API Handler internal/api/handlers/automations.go	Adds optional DryRun *bool to AutomationPayload and applies provided value when creating/updating automations.
Service Logic internal/services/automations/service.go	Introduces dry-run orchestration (separates dry-run vs live apply), adds helpers (ruleUsesCondition, dedupeHashes), pendingDeletion, recordDryRunActivities, run-item builders, and integrates dry-run recording without mutating state.
Service Tests internal/services/automations/service_test.go	Adds TestRecordDryRunActivities_Deletes to assert dry-run delete activity logging; imports time.
Frontend Types & Export/Import web/src/types/index.ts, web/src/lib/workflow-utils.ts	Adds dryRun to Automation/AutomationInput and WorkflowExport round-trip; includes "dry-run" in activity outcome union.
Workflow Dialog UI web/src/components/instances/preferences/WorkflowDialog.tsx	Adds dryRun to form state, Dry run toggle, localStorage-backed prompt when enabling workflows, and flows to preview/apply with dry-run.
Activity Run Dialog UI web/src/components/instances/preferences/AutomationActivityRunDialog.tsx	Displays "(dry run)" in dialog title and shows descriptive note when outcome is dry-run.
Workflows Overview UI web/src/components/instances/preferences/WorkflowsOverview.tsx	Adds formatCountWithVerb, formatDeleteDryRunSummary, updates formatters to accept optional outcome, renders dry-run badges/text, and includes dry-run in filtering logic.
Misc / Manifest internal/api/handlers/automations.go, package.json	Handler accepts dryRun in payload; minor manifest/test adjustments.

Sequence Diagram

sequenceDiagram
    actor User
    participant UI as WorkflowDialog
    participant API as API Handler
    participant Service as Automation Service
    participant ActivityStore as Activity Store
    participant DB as Database

    User->>UI: Toggle Enable + Dry run
    UI->>API: POST/PUT automation { dryRun: true }
    API->>Service: Create/Update + trigger apply flow
    Service->>Service: Split apply -> dry-run branch & live branch
    Service->>ActivityStore: recordDryRunActivities(pending items)
    ActivityStore->>DB: INSERT activities (outcome="dry-run")
    DB-->>ActivityStore: OK
    Service-->>API: return dry-run activity summary
    API-->>UI: Response with dry-run activities
    UI->>User: Display "(dry run)" badges and run details

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• feat(automations): show activity run details #1385 — Overlaps with automation activity/run recording and run-item builders used for dry-run reporting.
• feat: add workflow automation system with category actions, orphan scanner, and hardlink detection #818 — Prior automation changes that introduced core automation types and flows; this PR extends those models and handlers.
• fix(automations): speed up delete hardlink-copy expansion #1187 — Related service-level changes around delete/expand/preview handling that intersect with dry-run orchestration.

Suggested labels

api

Poem

🐰 I hopped through rules with cautious cheer,
I logged each step but kept things clear.
A gentle preview, no files gone astray,
I nibble the path and show the way.
Dry-run lights glow — safe as play.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 15.38% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'feat: add dry-run workflows' directly describes the main objective: adding dry-run support to workflow automations.
Linked Issues check	✅ Passed	The implementation fully addresses issue #1393: adds a dry-run toggle in the workflow editor [UI in WorkflowDialog.tsx], implements dry-run simulation without executing actions [via recordDryRunActivities and applyRulesForInstance], provides activity logging showing what would happen [ActivityOutcomeDryRun], and includes optional prompt on first enable [dryRunPrompt logic].
Out of Scope Changes check	✅ Passed	All changes are directly scoped to implementing dry-run workflows: database schema additions, backend service orchestration, activity tracking, and frontend UI controls. No unrelated modifications detected.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch codex/1122-dry-run-toggle

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 3

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

web/src/lib/workflow-utils.ts (1)

175-188: ⚠️ Potential issue | 🟠 Major

Preserve dryRun when parsing import JSON.
parseImportJSON never reads dryRun, so imported workflows lose the flag.

✅ Suggested fix

   // Optional intervalSeconds
   if (typeof obj.intervalSeconds === "number" && obj.intervalSeconds >= 60) {
     data.intervalSeconds = obj.intervalSeconds
   }
+
+  // Optional dryRun
+  if (typeof obj.dryRun === "boolean") {
+    data.dryRun = obj.dryRun
+  } else if (obj.dryRun !== undefined) {
+    return { data: null, error: "Invalid 'dryRun' field" }
+  }

web/src/types/index.ts (1)

356-379: ⚠️ Potential issue | 🔴 Critical

Remove duplicate programName field (TS2300).
programName is declared twice in the details object (lines 367 and 378), causing a TypeScript compilation error. Remove the second occurrence.

🛠️ Proposed fix

-    programName?: string

🤖 Fix all issues with AI agents

In `@internal/services/automations/service_test.go`:
- Around line 1336-1338: Replace the explicit empty-string equality check with
testify's empty assertion: change the assertion that compares
mockDB.activities[0].Hash to "" to use assert.Empty(t,
mockDB.activities[0].Hash) so the test uses the idiomatic empty check alongside
the existing require.Len(t, mockDB.activities, 1) and assert.Equal for Action
(models.ActivityActionDeletedCondition).

In `@internal/services/automations/service.go`:
- Around line 1519-1525: The file fails gofmt; reformat the affected block
around the dryRun check and the instLastApplied update to comply with gofmt
style: run `gofmt -w` (or `gofmt -w internal/services/automations/service.go`)
or use your editor's Go formatter so the code around the `if !dryRun { ...
s.mu.Lock(); instLastApplied[h] = now; s.mu.Unlock() }` block is properly
formatted; ensure no stray spacing or line breaks remain so the linter/pipeline
passes.
- Around line 1032-1049: The current sequence runs live rules before dry-run
rules causing live updates to instLastApplied and making dry-run rules skip
recent torrents; swap the calls so s.applyRulesForInstance(ctx, instanceID,
force, dryRunRules, true) runs before s.applyRulesForInstance(ctx, instanceID,
force, liveRules, false) to ensure dry-run rules evaluate all torrents
(reference variables liveRules, dryRunRules and function applyRulesForInstance,
and the instLastApplied/SkipWithin behavior).

🧹 Nitpick comments (1)

web/src/components/instances/preferences/WorkflowsOverview.tsx (1)

243-253: Consider simplifying the nested ternary for readability.

The deeply nested ternary operators can be harder to read and maintain. A mapping object would be cleaner:

♻️ Suggested refactor

 function formatDeleteDryRunSummary(details: AutomationActivity["details"], action: AutomationActivity["action"]): string {
   const count = details?.count ?? 0
-  const label = action === "deleted_ratio"
-    ? "ratio limit"
-    : action === "deleted_seeding"
-      ? "seeding limit"
-      : action === "deleted_unregistered"
-        ? "unregistered"
-        : "condition"
+  const labelMap: Record<string, string> = {
+    deleted_ratio: "ratio limit",
+    deleted_seeding: "seeding limit",
+    deleted_unregistered: "unregistered",
+    deleted_condition: "condition",
+  }
+  const label = labelMap[action] ?? "condition"
   return `${count} torrent${count !== 1 ? "s" : ""} would be deleted (${label})`
 }

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@web/src/components/instances/preferences/WorkflowsOverview.tsx`:
- Around line 187-253: The dry-run verbs should use passive voice for
consistency: update formatPausedSummary to set verb to "would be paused" when
outcome === "dry-run", update formatCategoryChangedSummary to set verb to "would
be moved" for dry-run, and update formatMovedSummary to use "would be moved" for
dry-run (keeping the existing failed branch using "failed to move"); these
changes are in the functions formatPausedSummary, formatCategoryChangedSummary,
and formatMovedSummary and rely on the existing formatCountWithVerb helper to
produce the final string.

[ShagoY]

Since pr-1362, when I try to go back to develop, I get the following error:
{ "level": "fatal", "error": "failed to run migrations: failed to apply migrations: failed to execute migration 058_add_automation_dry_run.sql: SQL logic error: duplicate column name: dry_run (1)", "time": "2026-02-08T19:02:35+01:00", "message": "Failed to initialize database" }

[s0up4200]

Since pr-1362, when I try to go back to develop, I get the following error: { "level": "fatal", "error": "failed to run migrations: failed to apply migrations: failed to execute migration 058_add_automation_dry_run.sql: SQL logic error: duplicate column name: dry_run (1)", "time": "2026-02-08T19:02:35+01:00", "message": "Failed to initialize database" }

Please dont run PRs like this without knowing the consequences.

Easiest fix for you right now is to shut down qui, delete qui.db + qui.db-shm and qui.db-wal. Then start qui again.
This means you have to reconnect and add your automations etc. again.

Alternatively you could try this:

Shut down qui.
Run:

sqlite3 -cmd ".timeout 5000" qui.db "PRAGMA table_info(automations);"
sqlite3 -cmd ".timeout 5000" qui.db "SELECT filename, applied_at FROM migrations WHERE filename LIKE '058%';"

If automations already has dry_run but no row for 058_add_automation_dry_run.sql, just mark it applied:

sqlite3 -cmd ".timeout 5000" qui.db \
"INSERT OR IGNORE INTO migrations (filename) VALUES ('058_add_automation_dry_run.sql');"

If you see some other 058 row then rename that row:

sqlite3 -cmd ".timeout 5000" qui.db \
"UPDATE migrations SET filename='058_add_automation_dry_run.sql' WHERE filename='058_add_dryrun.sql';"