Skip to content

Remove local scripts, fix workflows, and add test coverage#11

Merged
alari76 merged 1 commit intomainfrom
fix/remove-local-scripts
Mar 8, 2026
Merged

Remove local scripts, fix workflows, and add test coverage#11
alari76 merged 1 commit intomainfrom
fix/remove-local-scripts

Conversation

@alari76
Copy link
Copy Markdown
Contributor

@alari76 alari76 commented Mar 8, 2026

Summary

  • Remove local deploy/ops scripts (.codekin/scripts/) from the repo
  • Fix empty repo list in AddWorkflowModal (missing auth token)
  • Add repo-specific workflow discovery and dynamic workflow kinds API
  • Fix shell injection in workflow-loader and silent errors on workflow actions
  • Add comprehensive test coverage for workflow engine, loader, config, sessions, and helpers

Test plan

  • Verify npm test passes with new test suites
  • Open AddWorkflowModal and confirm repo list populates
  • Trigger a workflow run and verify it completes
  • Confirm deploy/ops scripts are removed from the repo

🤖 Generated with Claude Code

…epos

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alari76 alari76 merged commit 1921bd2 into main Mar 8, 2026
2 checks passed
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