Skip to content

fix(cloud): pipefail, exit codes via tee, HTML escape, rate limit (#137)#139

Closed
gHashTag wants to merge 1 commit into
mainfrom
feat/issue-137
Closed

fix(cloud): pipefail, exit codes via tee, HTML escape, rate limit (#137)#139
gHashTag wants to merge 1 commit into
mainfrom
feat/issue-137

Conversation

@gHashTag
Copy link
Copy Markdown
Owner

Closes #137

Fixes bugs from PR #133:

  • Change shebang from /bin/sh to /bin/bash for pipefail support
  • Add set -o pipefail for proper error handling in pipes
  • Fix Claude Code pipe with tee + PIPESTATUS to capture exit code correctly
  • Fix test pipe with tee + temp file to preserve exit code
  • Add HTML escaping for Telegram messages to handle < > & chars
  • Add rate limit protection (3s minimum) to send_telegram and stream_to_telegram

Fixes bugs from PR #133:
- Change shebang from /bin/sh to /bin/bash for pipefail support
- Add set -o pipefail for proper error handling in pipes
- Fix Claude Code pipe with tee + PIPESTATUS to capture exit code correctly
- Fix test pipe with tee + temp file to preserve exit code
- Add HTML escaping for Telegram messages to handle < > & chars
- Add rate limit protection (3s minimum) to send_telegram and stream_to_telegram

Closes #137
@gHashTag
Copy link
Copy Markdown
Owner Author

Closing — superseded by merged PRs #129, #130, #138. Reopen if needed.

@gHashTag gHashTag closed this Mar 11, 2026
gHashTag added a commit that referenced this pull request Mar 18, 2026
…, Multi-Hop 1000-Scale 100/100, Benchmarks+Noise 50/50 (310 queries 100%) [Golden Chain #139]

Tests 139-141: large-scale KG integration at 1000 entities.
- Test 139: 100 relations x 10 pairs = 1000 triples, forward 50/50, reverse 50/50, cross-rejection 50/50, 10 domains 10/10
- Test 140: 10 five-hop chains 50/50, cross-domain 2-hop 20/20, bundled pool 20/20, parallel 10/10
- Test 141: noise floor 0.013, signal 0.275, SNR 19.5x, 5% noise 10/10, 10% noise 10/10, replay+milestones 15/15
- Scale: 200 → 1000 entities with only 15% SNR degradation, zero accuracy loss
- Full regression: 413 tests, 409 pass, 4 skip, 0 fail

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@gHashTag gHashTag deleted the feat/issue-137 branch April 3, 2026 10:59
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.

fix(cloud): PR #133 bugs — subshell exit codes, pipefail, allowedTools

1 participant