Skip to content

Release Candidate 0.4.1#6

Open
fastfedora wants to merge 9 commits intomainfrom
releases/0.4.1
Open

Release Candidate 0.4.1#6
fastfedora wants to merge 9 commits intomainfrom
releases/0.4.1

Conversation

@fastfedora
Copy link
Owner

Adds do_while_item action and --max-tokens CLI option, along with several Codex environment fixes and CLI improvements.

Summary

New Features

  • Add do_while_item action, which executes actions at least once and then repeatedly while a condition is true. Unlike while_item, the condition is evaluated after the actions, so iteration is 1 on the first condition check.
  • Added --max-tokens CLI option (DF_MAX_TOKENS for env) to override the default token limit (8096), enabling longer model responses.

Enhancements

  • Pipeline parameters are now logged at startup, making it easier to see which parameters were actually used.
  • Refactored while_item to align its iteration-counting behavior with do_while_item.

Bug Fixes

  • Fixed Ctrl-C not working when --display is log. Both the full and log displays can now be cleanly exited.
  • Fixed setup scripts silently swallowing errors by adding -e flag, so failures are surfaced rather than ignored.
  • Fixed a permissions issue in the Codex Dockerfile where copying setup-repo.sh to /usr/local/bin required root.
  • Ensured pip and python are installed in the Codex Docker environment for repo setup scripts that expect them.
  • Ensured updated_at is set on first run so it is always present in item metadata.

While technically `updated_at` isn’t needed until the item has been actually updated, that makes testing the latest modification date much harder. Instead, ensure that `updated_at` always exists in the metadata.
We need to be root to copy into `/usr/local/bin` and do the `chmod`.
If the setup script didn’t enable the `-e` option, then errors encountered during setup were ignored and the setup script continued. But we want all errors to bubble up so they can be fixed by any pipeline that needs to fix them.
Same as `while_item` action, but the condition is evaluated after the actions rather than before.
Since `iterations` isn’t used before it is incremented, except in the debug statement, move it above the statement, so the statement doesn’t have to add one also.
Allow the number of max tokens to be overridden (default is 8096), so that longer responses can be received.
Ctrl-Q works to execute the full display, but Ctrl-C wasn’t working to exit the log display. This fixes it in a way such that both the full and log displays can be cleanly exited.
Make it easier to see which parameters were actually set.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant