Screeps/screeps-deploy-action
4
1
Fork 0
Code Issues 1 Pull Requests Actions Packages Releases 4 Activity

feat: implement fail-fast monitoring #86

Closed
Philipp wants to merge 6 commits from feature/fail-fast-monitor into main
pull from: feature/fail-fast-monitor
merge into: Screeps:main
Conversation 0 Commits 6 Files Changed 12
+3123 -17
Philipp commented 2026-05-16 18:29:30 +02:00
Owner
Copy Link
Copy Source

This PR implements fail-fast monitoring logic. If on_traceback, on_error_log, or on_warning_log is set to fail, the monitoring loop will terminate immediately upon detection of the respective event.

Changes:

  • Added shouldStop parameter to pollUntilDone in monitor.js.
  • Implemented fail-fast check in monitorConsole.
  • Added unit test to verify early exit behavior.
  • Rebuilt distribution files.
This PR implements fail-fast monitoring logic. If `on_traceback`, `on_error_log`, or `on_warning_log` is set to `fail`, the monitoring loop will terminate immediately upon detection of the respective event. Changes: - Added `shouldStop` parameter to `pollUntilDone` in `monitor.js`. - Implemented fail-fast check in `monitorConsole`. - Added unit test to verify early exit behavior. - Rebuilt distribution files.
Philipp added 5 commits 2026-05-16 18:29:30 +02:00
feat: add Screeps console monitoring with configurable error handling and shard support
Lint / pre-commit Linting (push) Successful in 59s
Details
Test / Run Tests (push) Successful in 1m47s
Details
172204508b
Merge branch 'main' into feature/console-monitor
Lint / pre-commit Linting (push) Successful in 1m3s
Details
Test / Run Tests (push) Successful in 1m49s
Details
84224f3678
chore: bump version to 0.2.0 242b03bb29
Merge branch 'feature/console-monitor' of git.horstenkamp.eu:Screeps/screeps-deploy-action into feature/console-monitor
Lint / pre-commit Linting (push) Successful in 45s
Details
Test / Run Tests (push) Successful in 1m11s
Details
55a9fc027d
feat: implement fail-fast monitoring to stop as soon as error/traceback is detected
Lint / pre-commit Linting (push) Successful in 42s
Details
Test / Run Tests (push) Successful in 1m3s
Details
b6cef04a9c
CODEOWNERS rules requested review from AutoReview 2026-05-16 18:29:30 +02:00
Philipp added 1 commit 2026-05-16 18:39:54 +02:00
fix: decode console output to ensure tracebacks are detected even when encoded
Lint / pre-commit Linting (push) Successful in 51s
Details
Test / Run Tests (push) Successful in 1m10s
Details
fe7c14540e
Philipp closed this pull request 2026-05-16 19:10:30 +02:00
All checks were successful
Lint / pre-commit Linting (push) Successful in 51s
Required
Details
Test / Run Tests (push) Successful in 1m10s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
AutoReview
Labels
Clear labels
bug
Something is not working
 chore
duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
AutoReview (LLMReview) Philipp (Philipp Horstenkamp) Renovate (Renovate Bot) runner (Act Runner)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Screeps/screeps-deploy-action#86
Delete Branch "feature/fail-fast-monitor"

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?