-
Notifications
You must be signed in to change notification settings - Fork 46
chore(release): 0.33.0 #1069
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(release): 0.33.0 #1069
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,19 @@ | ||
| ## 0.33.0 (2026-08-20) | ||
|
|
||
| ### BREAKING CHANGES | ||
| * The `total-disk-used-percent` metric now reports on a 0-100 scale instead of the incorrect 0.0-1.0 fraction. Previously, a 25% full disk was reported as 0.25; it is now correctly reported as 25. If you have alerts or dashboards based on this metric, update your thresholds accordingly. (#1067) | ||
| * Session working directory naming on Windows has changed (session ID prefix removed, `embedded_files` renamed to `ef`) for MAX_PATH compliance. This may affect workflows that depend on specific working directory paths. (#1068) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The dependency changes behind this release are not recorded, which breaks a pattern the previous release established. 0.33.0 changes two requirements in
0.32.0 documented exactly this kind of change as its own bullet — "Raised the openjd-model dependency floor to >= 0.11.3, which fixes: ..." (#1054) — so the omission here reads as an oversight rather than a deliberate style choice. Related: this entry describes the consequence of the |
||
|
|
||
| ### Features | ||
| * Added support for EXPR parameter types (`bool`, `rangeExpr`, `stringList`, `pathList`, `intList`, `floatList`, `boolList`, `intListList`) in API response parsing. Jobs using these parameter types will no longer crash the session. (#1064) | ||
| * The worker agent now accepts and forwards the `resolvedSymbolTable` from the service to the session runtime, enabling pre-resolved EXPR symbols (Job.Name, Param.*, RawParam.*, step let values) to be used in sessions. (#1063) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. "forwards the Per #1063, the Python adapter "accepts and ignores" the table for symbol resolution (v0 resolves EXPR natively); the only thing it pulls out is Since Suggest splitting the claim: "...On the Rust session runtime the table is forwarded to the session, enabling pre-resolved EXPR symbols (Job.Name, Param., RawParam., step let values). On the Python session runtime, Job.Name is seeded from the table; the remaining symbols continue to be resolved natively." |
||
| * Session extension enablement is now driven by the job's declaration in `JobDetails`. Jobs can explicitly declare which extensions they need, and the worker agent will enable only those extensions (plus `REDACTED_ENV_VARS`). (#1062) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is filed under Features, but as described it is the one entry in this release whose behavior can regress a working job without any change on the operator's side — it deserves a caveat here even if the classification stays. Before #1062, Per the comment on Two suggestions:
|
||
|
|
||
| ### Bug Fixes | ||
| * Fixed step-scoped environments not receiving their step's name and let bindings, which could cause incorrect environment configuration during sessions. (#1061) | ||
| * Fixed a crash when jobs use LIST parameter types (e.g., `LIST[STRING]`, `LIST[INT]`) due to incorrect Rust parameter type enum member lookup by value instead of name. (#1066) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This entry omits the two facts a reader needs to decide whether it affects them.
Suggest: "Fixed a crash under the Rust session runtime when jobs use |
||
| * Fixed EXPR parameter types being rejected during job entity validation, which prevented jobs using these parameter types from running via the BatchGetJobEntity path. (#1065) | ||
| * Fixed materialized embedded files on Windows not having correct permissions. An explicit Windows ACL is now set granting the agent user full control and the job user read access, preventing issues when NTFS inheritance is absent or misconfigured. (#1059) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. "granting the agent user full control" overstates the ACL that is actually set. The code in #1059 passes
So the agent gets read/write, not full control (no Also, the entry says "materialized embedded files on Windows," which reads as covering all embedded-file materialization. The change is scoped to the Rust session runtime adapter's stopgap materialization path ( Suggest: "...An explicit Windows ACL is now set granting the agent user read/write and the job user read access on files materialized by the Rust session runtime..." |
||
| ## 0.32.0 (2026-08-14) | ||
|
|
||
| ### BREAKING CHANGES | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "previously reported as 0.25" example is not what the old code actually emitted.
The pre-fix expression was
round(disk.used / disk.total, ndigits=1)— rounding to one decimal place. So a 25%-full disk was emitted as0.2, not0.25. The commit message for #1067 states this correctly ("A 25% full disk was reported as 0.2") and the old unit test asserted== "0.2", so this looks like a transcription slip when the note was written.This matters more than a typo here, because the whole point of the entry is to help operators re-tune alarms: someone reconstructing their old thresholds from
0.25will assume the old series had two-digit precision, when in reality every value was quantized to 0.1 (i.e. only 11 distinct values ever appeared). Suggest: