Update skills with CI lessons learned #9

Merged

despiegk merged 6 commits from development_skills_update into development

2026-02-17 01:12:15 +00:00

mik-tf commented

2026-02-16 20:45:37 +00:00

Owner

Document CI pitfalls and fixes discovered while fixing hero_lib CI pipeline.

Changes

naming_convention: Fix Cargo 1.92 normalization claim — -p flag now requires exact package name
oschema_generation: Add warning that *_generated.rs files must not be manually edited
forge_ci: Add DEBIAN_FRONTEND, ((VAR++)) pitfall, if:false gotcha, linker deps, cargo_env note, fix token docs
build_lib: Add cargo_env explicitness requirement, bash arithmetic pitfall, DEBIAN_FRONTEND
hero_ecosystem: Add system library requirements for herolib_virt (libseccomp-dev, libcap-ng-dev)

Testing

All learnings verified during hero_lib CI fix session (PR lhumina_code/hero_lib#104).

Closes #8

Document CI pitfalls and fixes discovered while fixing hero_lib CI pipeline. ## Changes - **naming_convention**: Fix Cargo 1.92 normalization claim — `-p` flag now requires exact package name - **oschema_generation**: Add warning that `*_generated.rs` files must not be manually edited - **forge_ci**: Add DEBIAN_FRONTEND, `((VAR++))` pitfall, `if:false` gotcha, linker deps, cargo_env note, fix token docs - **build_lib**: Add cargo_env explicitness requirement, bash arithmetic pitfall, DEBIAN_FRONTEND - **hero_ecosystem**: Add system library requirements for herolib_virt (libseccomp-dev, libcap-ng-dev) ## Testing All learnings verified during hero_lib CI fix session (PR lhumina_code/hero_lib#104). Closes #8

mik-tf added 5 commits

2026-02-16 20:45:37 +00:00

docs: fix Cargo normalization claim in naming_convention ba340e8f2b

Cargo 1.92 no longer normalizes hyphens/underscores for the -p flag.
`cargo build -p herolib-do` fails when the package is `herolib_do`.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: add generated file warning to oschema_generation b953e6ec4a

Clarify that *_generated.rs files are overwritten on every build.
Fix issues in the generator (rust_osis.rs), not in generated output.

Refs #8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: add CI pitfalls to forge_ci skill 6cd83f7f59

- DEBIAN_FRONTEND=noninteractive for apt-get in Docker
- ((VAR++)) under set -e returns exit code 1 when VAR=0
- if:false still shows pending status in Forgejo
- libseccomp-dev/libcap-ng-dev for virtiofsd linking
- cargo_env must be called explicitly after sourcing
- Fix FORGEJO_TOKEN documentation

Refs #8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: add CI pitfalls and cargo_env note to build_lib dbcca19241

- cargo_env must be called explicitly after sourcing build_lib.sh
- ((VAR++)) returns exit code 1 when VAR=0 under set -e
- DEBIAN_FRONTEND=noninteractive for apt-get in Docker

Refs #8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: add system library requirements to hero_ecosystem 70e3a01eca

herolib_virt (virtiofsd) requires libseccomp-dev and libcap-ng-dev
in CI Docker images for linking.

Refs #8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

despiegk added 1 commit

2026-02-17 01:12:03 +00:00