Issue #24 successfully migrated all shell script functionality to the TypeScript client. The webhook server now calls the TypeScript client directly via commands.json.
Remaining Cleanup Tasks
Remove obsolete shell script:
Delete scripts/handle-issue-ts.sh (no longer used)
Update documentation files:
commands.json.example - Update issue/comment handler examples to show TypeScript client usage (not shell script)
README.md - Remove references to the shell script approach
Current Status
✅ Migration complete - TypeScript client is fully functional
✅ commands.json already uses TypeScript client directly
❌ Old shell script still exists in repository
❌ Documentation still references shell script approach
Expected Outcome
Clean repository without obsolete scripts
Documentation accurately reflects current TypeScript-based implementation
Example files show best practices using TypeScript client
## Background
Issue #24 successfully migrated all shell script functionality to the TypeScript client. The webhook server now calls the TypeScript client directly via `commands.json`.
## Remaining Cleanup Tasks
1. **Remove obsolete shell script**:
- Delete `scripts/handle-issue-ts.sh` (no longer used)
2. **Update documentation files**:
- `commands.json.example` - Update issue/comment handler examples to show TypeScript client usage (not shell script)
- `README.md` - Remove references to the shell script approach
## Current Status
✅ Migration complete - TypeScript client is fully functional
✅ `commands.json` already uses TypeScript client directly
❌ Old shell script still exists in repository
❌ Documentation still references shell script approach
## Expected Outcome
- Clean repository without obsolete scripts
- Documentation accurately reflects current TypeScript-based implementation
- Example files show best practices using TypeScript client
Delete the obsolete scripts/handle-issue-ts.sh file
Update commands.json.example to show TypeScript client usage
Update README.md to remove shell script references
Once completed, I'll commit the changes, push them, and close this issue as requested.
@fszontagh I'm starting work on this issue now!
I'll complete the following cleanup tasks:
1. Delete the obsolete `scripts/handle-issue-ts.sh` file
2. Update `commands.json.example` to show TypeScript client usage
3. Update `README.md` to remove shell script references
Once completed, I'll commit the changes, push them, and close this issue as requested.
Changed claude-issue-handler to use type: "node" and call /data/agent-manager/client/dist/index.js directly
Changed claude-comment-handler to use type: "node" and call /data/agent-manager/client/dist/index.js directly
Removed cwd: "~" and changed to cwd: null (using project root)
Updated README.md: Removed shell script reference from project structure section
All issue/comment handlers now invoke the TypeScript client directly as Node.js processes, eliminating the shell script wrapper. The documentation accurately reflects the current TypeScript-based implementation.
✅ Cleanup complete! All tasks have been finished:
1. **Deleted obsolete shell script**: Removed `scripts/handle-issue-ts.sh`
2. **Updated `commands.json.example`**:
- Changed `claude-issue-handler` to use `type: "node"` and call `/data/agent-manager/client/dist/index.js` directly
- Changed `claude-comment-handler` to use `type: "node"` and call `/data/agent-manager/client/dist/index.js` directly
- Removed `cwd: "~"` and changed to `cwd: null` (using project root)
3. **Updated `README.md`**: Removed shell script reference from project structure section
All issue/comment handlers now invoke the TypeScript client directly as Node.js processes, eliminating the shell script wrapper. The documentation accurately reflects the current TypeScript-based implementation.
**Commit**: `43568b5` - "chore: remove obsolete shell script and update documentation #26"
Closing this issue as requested.
Claude Coding Agent
Background
Issue #24 successfully migrated all shell script functionality to the TypeScript client. The webhook server now calls the TypeScript client directly via
commands.json.Remaining Cleanup Tasks
Remove obsolete shell script:
scripts/handle-issue-ts.sh(no longer used)Update documentation files:
commands.json.example- Update issue/comment handler examples to show TypeScript client usage (not shell script)README.md- Remove references to the shell script approachCurrent Status
✅ Migration complete - TypeScript client is fully functional ✅
commands.jsonalready uses TypeScript client directly ❌ Old shell script still exists in repository ❌ Documentation still references shell script approachExpected Outcome
do this. If job is done close this issue.
@fszontagh I'm starting work on this issue now!
I'll complete the following cleanup tasks:
scripts/handle-issue-ts.shfilecommands.json.exampleto show TypeScript client usageREADME.mdto remove shell script referencesOnce completed, I'll commit the changes, push them, and close this issue as requested.
✅ Cleanup complete! All tasks have been finished:
scripts/handle-issue-ts.shcommands.json.example:claude-issue-handlerto usetype: "node"and call/data/agent-manager/client/dist/index.jsdirectlyclaude-comment-handlerto usetype: "node"and call/data/agent-manager/client/dist/index.jsdirectlycwd: "~"and changed tocwd: null(using project root)README.md: Removed shell script reference from project structure sectionAll issue/comment handlers now invoke the TypeScript client directly as Node.js processes, eliminating the shell script wrapper. The documentation accurately reflects the current TypeScript-based implementation.
Commit:
43568b5- "chore: remove obsolete shell script and update documentation #26"Closing this issue as requested.