lsm / neokai / 24921456527
82%

DEFAULT BRANCH: dev
Ran
Jobs 0
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
24921456527

push

github

web-flow 
chore(space): remove completionActions types + schema + docs (PR 5/5)

* chore(space): remove completionActions types + schema (PR 5/5)

Final cleanup of the task-agent-as-post-approval-executor refactor. PRs 1–4
moved every runtime path off completionActions; this stage deletes the
types, schema columns, and supporting code.

Shared types
- Drop CompletionAction union (ScriptCompletionAction,
  InstructionCompletionAction, McpCallCompletionAction) and the
  completionActions field on WorkflowNode.
- Drop pendingActionIndex from SpaceTask and completionActionsFiredAt
  from SpaceWorkflowRun.
- Narrow pendingCheckpointType to 'gate' | 'task_completion'.

Daemon
- space-workflow-handlers, space-workflow-manager, template-hash,
  space-task-repository, space-workflow-run-repository, and
  m94-backfill-workflow-templates no longer thread or persist
  completion-action state.
- space-workflow-repository.rowToNode now strips an unknown
  completionActions key from legacy node config rows and emits a single
  structured workflow.migrated log line per getWorkflow / listWorkflows
  call (plan §6.1).

Migration 104 (PR 5/5 schema cleanup)
- Defensively rewrites any in-flight pending_checkpoint_type='completion_action'
  rows to 'task_completion' and clears pending_action_index.
- Rebuilds space_tasks to drop pending_action_index and tighten the
  CHECK to ('gate', 'task_completion') (mirrors the M99/M103 rebuild
  pattern, preserves indexes).
- ALTER TABLE space_workflow_runs DROP COLUMN completion_actions_fired_at.
- Idempotent and guarded for missing-table cases.

Tests
- New migration-104_test.ts covering fresh-DB, table-rebuild with stuck
  rows, dropped columns, tightened CHECK, idempotency, and missing-table
  guards (16 cases).
- Updated migration-94_test.ts to drop merge-pr injection assertions and
  replaced the now-irrelevant "row already backfilled is left alone" test
  with a "node config JSON is never rewritten" test.
- Updated migration-spaces-aut... (continued)
Source Files on build 24921456527
Detailed source file information is not available for this build.