Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
bf0c930 to
91e22da
Compare
8ff39bb to
f43bd45
Compare
|
@greptileai The stack has been updated with the shared contract and startup-ownership fixes. The remaining API/runtime/CLI findings are still being addressed; their completed fixes and regression evidence will be sent for another review. This update does not claim those findings are resolved. |
f43bd45 to
dc869ce
Compare
91e22da to
9779158
Compare
|
@greptileai Please review the current head after the stack update. The previous findings have been addressed with fixes and regression coverage, or an evidence-backed reply where local hardware initialization was mistaken for paid infrastructure allocation. Every review thread has a concrete response. Please check the current diff and report any remaining defect. |
9779158 to
93cbce2
Compare
dc869ce to
79857d3
Compare
|
@greptileai Please review the current head. This stack update only corrects and strengthens the CUDA checkpoint assertion, with six CPU corruption cases; public runtime code is unchanged. The previously addressed findings remain fixed. |
93cbce2 to
5e6e34d
Compare
79857d3 to
fe48926
Compare
|
@greptileai Please review this current head after rebasing onto main's native0.3.81 integer Responses timestamps. The test conflict preserves all new main assertions and independent fixture isolation. CLaaS runtime source is unchanged by the rebase. Prior findings remain addressed. |
fe48926 to
c5340f8
Compare
5e6e34d to
e1ab37e
Compare
|
@greptileai Please review the current head. The security findings are fixed with regression coverage: retained unreaped session identity, no raw group-kill fallback, and fixed /usr/bin/sync for Modal commits. Previous review findings remain addressed. The focused final cleanup/Modal/CLI suite passed66 tests; full current-head CI is running. |
Expose exp optimize claas burst CONFIG and exp optimize claas serve CONFIG, locally or on Modal. Both validate bounded configuration/imports and use shared spend consent before selecting compute. Local execution uses the process supervisor; Modal foreground ownership retains exact recovery identity and bounds interruption cleanup.
The learning workflow reuses the existing AgentRuntime/EnvironmentRuntime loop and collects response IDs for explicit delayed feedback. Failure mining, world-model scenarios, evaluation, and deployment stay outside CLaaS. An explicit zero estimate declares nonbillable existing local hardware; metered hosts require their conservative estimate and Modal always requires a positive estimate.
Validation: actual CLI help and empty-burst process execution, real SDK/API/scaffold integration, cancellation/recovery, release checks, and dependency isolation. Final full local suite: 4,945 passed, 8 explicit skips; whole Ruff/format/ty pass. Both real L4 backend integration tests pass; see #1021 for their scope.
Stack 7/7: #1018 → #1019 → #1020 → #1021 → #1022 → #1023 → this PR. No merge or production deployment requested.
The real local CLI also closes an empty burst successfully. The actual Modal launcher passes a live isolated empty-burst smoke with committed receipt/SQLite validation and verified cleanup.