Skip to content

wiring reads a changes job's paths-filter as a pin trigger, so ci.yml's corpus.version rebuild is listed #169

Closed

Description

@brunogbv

What

wiring.py reads a workflow's pin triggers only from on.push.paths (:210, :349). marola-dev/marola-app#128 moved the image build from docker.yml (which had paths: [corpus.version]) into ci.yml, which runs on every push and gates the image jobs with a changes job (dorny/paths-filter, filter docker, corpus.version among its paths). The image still rebuilds on a corpus bump, but the umbrella's REPOS.md wiring block lost the row marola-app corpus.version | marola-app ci.yml (marola-dev/marola#855).

Read a dorny/paths-filter step's filters: too, and count a pin as a trigger of that workflow when a filter that lists it gates a job (needs.changes.outputs.<filter> in an if:).

Acceptance criteria

  • wiring lists marola-app corpus.version | marola-app ci.yml in the Pin bump table at marola-app's current main
  • a filter whose output gates no job adds no row
  • just quality green; after the release and the umbrella bump, the umbrella's wiring --check passes with the regenerated row

Named test

scripts/wiring.py --self-test, with a fixture ci.yml that has a changes job filtering corpus.version and a job gated on it.


Follow-up to marola-dev/marola#855. Parent: marola-dev/marola#787.

Activity

  1. added
    area/dev-toolingClaude Code / OpenCode / agentic-tooling dev workflow, not the product
    layer/infra.github/, scripts/, justfile, flake.nix, .githooks/, .claude/
    size/MRoughly 100-400 changed lines
    on Oct 17, 2026
  2. added
    agent-readyPasses the five-rule Definition of Ready — an agent may begin work on it (MIP-0063 §5.4)
    and removed
    agent-readyPasses the five-rule Definition of Ready — an agent may begin work on it (MIP-0063 §5.4)
    on Oct 17, 2026
  3. self-assigned this
    on Oct 17, 2026
  4. moved this from Backlog to In progress in Marolaon Oct 17, 2026
  5. added 2 commits that reference this issue on Oct 17, 2026
    b089896
    706e2ee
  6. moved this from In progress to Done in Marolaon Oct 17, 2026
  7. added a commit that references this issue on Oct 17, 2026
  8. brunogbv commented on Oct 17, 2026

    @brunogbv
    CollaboratorAuthor

    Proof, 2026-10-17. Shipped in marola-devkit v0.17.0 (PR #173, release #175). The umbrella's bump marola-dev/marola#872, opened and merged by marola-merge-bot at 12:30:52Z, regenerated REPOS.md's wiring block with the new tag. Its Pin bump table on umbrella main (docs/2-Building-marola/REPOS.md:102-103) now has:

    | marola `flake.lock` | marola `ci.yml`, marola `docs.yml` |
    | marola-app `corpus.version` | marola-app `ci.yml` |
    

    The first row is the AC; the second is the umbrella's own ci.yml, whose docs, nix and wiring filters list flake.lock and gate docs-build and quality-other. Umbrella main CI on that commit, which runs wiring --check: run 39032827542, success (quality-other and ci-ok green).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area/dev-toolingClaude Code / OpenCode / agentic-tooling dev workflow, not the productlayer/infra.github/, scripts/, justfile, flake.nix, .githooks/, .claude/size/MRoughly 100-400 changed lines

Type

No type

Projects

  • Status
    Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions