cal/claude-home
1
0
Fork 0
Code Issues 6 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

feat: session resumption + Agent SDK evaluation #43

Merged
cal merged 1 commits from feature/3-agent-sdk-improvements into main 2026-04-03 20:00:14 +00:00
Conversation 0 Commits 1 Files Changed 3 +246 -4
cal commented 2026-04-03 19:57:13 +00:00
Owner
Copy Link

Summary

  • Session resumption (--resume): Added opt-in support to runner.sh via session_resumable and resume_last_session settings. Captures session_id in issue-poller.sh and pr-reviewer-dispatcher.sh logs/results. Updated n8n workflow with Follow Up Diagnostics node that resumes sessions for deeper investigation.
  • --append-system-prompt: Already implemented in all live scripts (confirmed). Added to legacy n8n workflow SSH node.
  • Agent SDK evaluation: Research doc comparing CLI vs Python/TS SDK. Recommendation: stay on CLI (SDK requires API key billing; we use Claude Max OAuth).
  • Bug fix: Normalized Python boolean output in read_setting() so JSON true maps to bash "true" (not "True").

Script changes committed separately to cal/dotfiles (main branch).

Closes #3

Test plan

  • runner.sh with session_resumable: true — --no-session-persistence absent, last_session_id saved
  • runner.sh with resume_last_session: true — --resume <id> present in command
  • Existing tasks without new settings — backward compat, --no-session-persistence still used
  • issue-poller.sh --dry-run — completes without errors
  • bash -n syntax check passes on all 3 scripts
  • n8n workflow JSON validates cleanly
  • n8n workflow import and manual trigger (requires CT 210/300)

🤖 Generated with Claude Code

## Summary - **Session resumption (`--resume`)**: Added opt-in support to runner.sh via `session_resumable` and `resume_last_session` settings. Captures `session_id` in issue-poller.sh and pr-reviewer-dispatcher.sh logs/results. Updated n8n workflow with Follow Up Diagnostics node that resumes sessions for deeper investigation. - **`--append-system-prompt`**: Already implemented in all live scripts (confirmed). Added to legacy n8n workflow SSH node. - **Agent SDK evaluation**: Research doc comparing CLI vs Python/TS SDK. Recommendation: stay on CLI (SDK requires API key billing; we use Claude Max OAuth). - **Bug fix**: Normalized Python boolean output in `read_setting()` so JSON `true` maps to bash `"true"` (not `"True"`). Script changes committed separately to `cal/dotfiles` (main branch). Closes #3 ## Test plan - [x] `runner.sh` with `session_resumable: true` — `--no-session-persistence` absent, `last_session_id` saved - [x] `runner.sh` with `resume_last_session: true` — `--resume <id>` present in command - [x] Existing tasks without new settings — backward compat, `--no-session-persistence` still used - [x] `issue-poller.sh --dry-run` — completes without errors - [x] `bash -n` syntax check passes on all 3 scripts - [x] n8n workflow JSON validates cleanly - [ ] n8n workflow import and manual trigger (requires CT 210/300) 🤖 Generated with [Claude Code](https://claude.com/claude-code)
cal added 1 commit 2026-04-03 19:57:14 +00:00
feat: add session resumption and Agent SDK evaluation
All checks were successful
Auto-merge docs-only PRs / auto-merge-docs (pull_request) Successful in 3s
Details
0dae52441a 
- runner.sh: opt-in session persistence via session_resumable and
  resume_last_session settings; fix read_setting to normalize booleans
- issue-poller.sh: capture and log session_id from worker invocations,
  include in result JSON
- pr-reviewer-dispatcher.sh: capture and log session_id from reviews
- n8n workflow: add --append-system-prompt to initial SSH node, add
  Follow Up Diagnostics node using --resume for deeper investigation,
  update Discord Alert with remediation details
- Add Agent SDK evaluation doc (CLI vs Python/TS SDK comparison)
- Update CONTEXT.md with session resumption documentation

Closes #3

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
cal force-pushed feature/3-agent-sdk-improvements from 0dae52441a to e321e7bd47 2026-04-03 19:59:48 +00:00 Compare
cal merged commit 13483157a9 into main 2026-04-03 20:00:14 +00:00
cal deleted branch feature/3-agent-sdk-improvements 2026-04-03 20:00:14 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
ai-changes-requested

   
ai-failed

   
ai-pr-opened

   
ai-reviewed

   
ai-reviewing

   
ai-working

   
infra-audit

Infrastructure audit remediation tasks

  
monitoring

Monitoring, alerting, and observability

  
operations

Ongoing operational improvements

  
proxmox

Proxmox VM/CT management

  
script

Script improvements and tooling

No Label
ai-changes-requested
ai-failed
ai-pr-opened
ai-reviewed
ai-reviewing
ai-working
infra-audit
monitoring
operations
proxmox
script
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/claude-home#43
No description provided.
Delete Branch "feature/3-agent-sdk-improvements"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?