umputun / ralphex

81%

master: 81%

LAST BUILD ON BRANCH add-notifications

branch: add-notifications

CHANGE BRANCH

x

Reset

• add-notifications
• add-default-branch-template
• add-dirty-worktree-check
• add-reset-flag
• auto-create-initial-commit
• auto-plan-mode-detection
• code-review-fixes
• commented-defaults
• configurable-colors
• custom-external-review
• dashboard-tests
• dependabot/go_modules/github.com/go-git/go-billy/v5-5.7.0
• dependabot/go_modules/golang.org/x/crypto-0.45.0
• display-lines-changed-stats
• docker-support
• error-patterns
• external-git-backend
• feat/agent-frontmatter-model-v2
• feat/per-phase-claude-models-v2
• feat/scalar-config-fallback
• feature/claude-plugin-support
• finalize-step
• fix-checkout-keep-untracked
• fix-codex-not-installed
• fix-codex-stderr-reporting
• fix-isignored-global-patterns
• fix-plan-file-resolution
• fix-process-group-cleanup
• fix-review-loop-exit
• fix-scanner-buffer-size
• fix-sigint-handling
• fix-uncommitted-changes-handling
• fix-version-unknown
• fix-windows-build
• fix-worktree-support
• fix/strip-comments-markdown
• force-exit-timeout
• graceful-prompt-variables
• interactive-plan-creation
• master
• plan-draft-preview
• project-local-config
• refactor-git-service
• refactor-main
• refs/tags/v0.1.0
• refs/tags/v0.10.0
• refs/tags/v0.10.1
• refs/tags/v0.10.2
• refs/tags/v0.10.3
• refs/tags/v0.10.4
• refs/tags/v0.10.5
• refs/tags/v0.2.0
• refs/tags/v0.2.1
• refs/tags/v0.2.2
• refs/tags/v0.2.3
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.4.3
• refs/tags/v0.4.4
• refs/tags/v0.5.0
• refs/tags/v0.6.0
• refs/tags/v0.7.0
• refs/tags/v0.7.1
• refs/tags/v0.7.2
• refs/tags/v0.7.3
• refs/tags/v0.7.4
• refs/tags/v0.7.5
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• remove-docker
• rewrite-dk-script-python
• web-fixes
• web-planner
• web-ui
• web-ui-improvements

Build # 21768579257

Build Type

Pull #71

github

Committed by umputun

Commit Message

fix(notify): address Copilot review findings on PR #71

- use context.Background() for success notification (parent ctx may be canceled)
- capture stdout in custom script executor error messages
- comment out notify_on_error/notify_on_complete in defaults config
- add code-level defaults for notify flags when not explicitly set

Pull Request Pull Request #71: feat: add notification system

Run Details

4867 of 6029 relevant lines covered (80.73%)

159.01 hits per line