didericis/bot-bottle
1
0
Fork 0
Code Issues 33 Pull Requests 9 Actions Packages Releases Wiki Activity

fix: quality cleanup - bounded heavy operations (chunk 10/11) #531

Closed
didericis-codex wants to merge 1 commits from fix/bounded-heavy-operations into fix/cleanup-execution-integrity
pull from: fix/bounded-heavy-operations
merge into: didericis:fix/cleanup-execution-integrity
Conversation 0 Commits 1 Files Changed 4
+95 -32
didericis-codex commented 2026-07-27 00:18:17 -04:00
Collaborator
Copy Link
Copy Source

Stacked on #530. Tenth quality-cleanup chunk tracked by #444 and PRD 0082.

  • Git smart-HTTP bodies spool to disk rather than accumulating up to 100 MiB per worker in memory.
  • A separate two-slot semaphore bounds concurrent body-heavy Git operations independently of lightweight request threads.
  • Shared cleanup subprocesses use a configurable BOT_BOTTLE_CLEANUP_COMMAND_TIMEOUT_SECONDS deadline (120 seconds by default) and aggregate timeouts as mutation failures.
  • Firecracker signalling failures aggregate and allow other approved mutations to run; identity-verification uncertainty still aborts fail-closed.

Validation: 49 focused cleanup/boundary tests pass; full-repository Pyright reports 0 errors; affected-module Pylint scores 9.92/10. The complete eleven-PR stack passes all 2,520 unit tests; full-repository Pyright reports 0 errors; git diff --check passes.

Refs #444.

Final stack extension

  • #532 — fail-closed shared database and gateway credential permissions.
Stacked on #530. Tenth quality-cleanup chunk tracked by #444 and PRD 0082. - Git smart-HTTP bodies spool to disk rather than accumulating up to 100 MiB per worker in memory. - A separate two-slot semaphore bounds concurrent body-heavy Git operations independently of lightweight request threads. - Shared cleanup subprocesses use a configurable `BOT_BOTTLE_CLEANUP_COMMAND_TIMEOUT_SECONDS` deadline (120 seconds by default) and aggregate timeouts as mutation failures. - Firecracker signalling failures aggregate and allow other approved mutations to run; identity-verification uncertainty still aborts fail-closed. **Validation:** 49 focused cleanup/boundary tests pass; full-repository Pyright reports 0 errors; affected-module Pylint scores 9.92/10. The complete eleven-PR stack passes all 2,520 unit tests; full-repository Pyright reports 0 errors; `git diff --check` passes. Refs #444. **Final stack extension** - #532 — fail-closed shared database and gateway credential permissions.
didericis added 1 commit 2026-07-27 00:47:17 -04:00
fix: bound heavy gateway operations
test / image-input-builds (pull_request) Failing after 11m25s
Details
test / unit (pull_request) Has started running
Details
test / integration-docker (pull_request) Blocked by required conditions
Details
test / coverage (pull_request) Blocked by required conditions
Details
tracker-policy-pr / check-pr (pull_request) Successful in 12s
Details
dbbb185d0a
didericis force-pushed fix/bounded-heavy-operations from 968d6c7606 to dbbb185d0a 2026-07-27 00:47:17 -04:00 Compare
didericis-codex changed title from fix: quality cleanup - bounded heavy operations (chunk 10/10) to fix: quality cleanup - bounded heavy operations (chunk 10/11) 2026-07-27 00:51:03 -04:00
didericis-codex closed this pull request 2026-07-27 01:36:06 -04:00
Some checks are pending
test / image-input-builds (pull_request) Failing after 11m25s
Details
test / unit (pull_request) Has started running
Details
test / integration-docker (pull_request) Blocked by required conditions
Details
test / coverage (pull_request) Blocked by required conditions
Details
tracker-policy-pr / check-pr (pull_request) Successful in 12s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Status/Needs Triage
Awaiting initial classification
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
didericis-claude (didericis (claude)) didericis-codex (didericis (codex)) didericis-gemma (gemma) didericis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: didericis/bot-bottle#531
Delete Branch "fix/bounded-heavy-operations"

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?