[limen GEN-organvm-iv-taxis-.github-ci-green-0620] Make organvm-iv-taxis/.github CI green#11
Conversation
limen task GEN-organvm-iv-taxis-.github-ci-green-0620
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b5c9338359
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| run: | | ||
| if [ -f seed.yaml ]; then | ||
| python3 -c "import yaml; yaml.safe_load(open('seed.yaml'))" && echo "seed.yaml is valid YAML" || { echo "::error::seed.yaml is invalid YAML"; exit 1; } | ||
| if ruby -e "require 'yaml'; YAML.load_file('seed.yaml')"; then |
There was a problem hiding this comment.
In the Validate seed.yaml step, this call only loads the first YAML document; Psych's own load docs state that when multiple documents are provided, the first document is returned. That means a seed.yaml like a: 1\n---\n: : : now exits 0 and prints that the file is valid even though later content is syntactically broken, whereas the previous single-document PyYAML load rejected extra/invalid documents. Use a parser that walks the full stream, such as YAML.parse_stream, or explicitly reject additional documents.
Useful? React with 👍 / 👎.
| run: | | ||
| if [ -f seed.yaml ]; then | ||
| python3 -c "import yaml; yaml.safe_load(open('seed.yaml'))" && echo "seed.yaml is valid YAML" || { echo "::error::seed.yaml is invalid YAML"; exit 1; } | ||
| if ruby -e "require 'yaml'; YAML.load_file('seed.yaml')"; then |
There was a problem hiding this comment.
Because this validates by deserializing with YAML.load_file instead of doing a parse-only check, valid YAML that uses anchors/aliases (including merge keys) or unquoted timestamp/date scalars now fails before the success message: Psych's safe loader disables aliases and does not permit Date/Time classes by default. In repos that add anchors to avoid repeated seed metadata or leave YAML dates unquoted, this workflow blocks otherwise valid seed.yaml files that the previous PyYAML validation accepted.
Useful? React with 👍 / 👎.
4444J99
deleted the
limen/gen-organvm-iv-taxis-.github-ci-green-0620-ecee
branch
1 participant
Autonomous limen dispatch of task
GEN-organvm-iv-taxis-.github-ci-green-0620.Inspect the latest FAILING checks on organvm-iv-taxis/.github's default branch, fix the root cause (lint / types / failing test / config), and confirm the checks pass. If CI is already green, add the single most valuable missing check (e.g. typecheck or test-matrix) instead. [auto-generated 2026-06-20 to keep the stream endless]
Produced in an isolated worktree off origin — review before merge.