Skip to content

AD-502: Add safe shell runtime adapter#40

Open
alph-cmky wants to merge 1 commit into
feat/ad-501-runtime-adapter-interfacefrom
feat/ad-502-safe-shell-adapter
Open

AD-502: Add safe shell runtime adapter#40
alph-cmky wants to merge 1 commit into
feat/ad-501-runtime-adapter-interfacefrom
feat/ad-502-safe-shell-adapter

Conversation

@alph-cmky

Copy link
Copy Markdown
Owner

Summary

  • Adds a Safe Shell runtime adapter implementing the shared RuntimeAdapter contract.
  • Allows only configured safe commands by default.
  • Rejects arbitrary commands before execution unless allowArbitraryCommands is explicitly enabled.
  • Streams stdout and stderr as task events and reports completion/failure events.
  • Exports shell adapter APIs from @agentdeck/runtime-adapters.
  • Marks AD-502 complete in the development backlog.

Stack

Verification

  • TDD red check: pnpm test packages/runtime-adapters/src/execution/shell-adapter.test.ts initially failed because shell-adapter.js did not exist yet.
  • pnpm test packages/runtime-adapters/src/execution/shell-adapter.test.ts
  • pnpm lint && pnpm format && pnpm typecheck && pnpm test && pnpm build

Closes #15

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