Skip to content

skills-update drops its token secret fallback (pre-MIP-0086) #148

Closed

Description

@brunogbv

What

.github/workflows/skills-update.yml still declares an optional token secret and reads
steps.app.outputs.token || secrets.token || github.token (:31, :75). #132 removed the same
fallback from notify-umbrella and gemini-review. No caller passes token: to skills-update today (marola-site passes
pr_bot_key; marola passes nothing and gets GITHUB_TOKEN), so the removal breaks no one, and GITHUB_TOKEN stays the
no-key path. Remove the secret and the middle fallback; update the docs that list it and add a CHANGELOG line
(breaking only for a caller that passes token:).

Acceptance criteria

  • grep -n 'secrets.token' .github/workflows/skills-update.yml prints nothing
  • docs and CHANGELOG updated; just quality green

Named test

just quality; a skills-update run in a consumer after the bump (run link).

Found while reviewing #132. Parent: marola-dev/marola#787. Not a blocker of #765 (the fallback ends at GITHUB_TOKEN).

Activity

  1. added
    enhancementNew feature or request
    area/dev-toolingClaude Code / OpenCode / agentic-tooling dev workflow, not the product
    layer/infra.github/, scripts/, justfile, flake.nix, .githooks/, .claude/
    size/SUnder ~100 changed lines
    on Oct 16, 2026
  2. added
    agent-readyPasses the five-rule Definition of Ready — an agent may begin work on it (MIP-0063 §5.4)
    on Oct 16, 2026
  3. moved this from Backlog to In progress in Marolaon Oct 17, 2026
  4. moved this from In progress to Done in Marolaon Oct 17, 2026
  5. added a commit that references this issue on Oct 17, 2026
  6. brunogbv commented on Oct 17, 2026

    @brunogbv
    CollaboratorAuthor

    Proof, 2026-10-17. Shipped in marola-devkit v0.17.0 (PR #170, release #175, tag on b804e73). marola-site's bump to v0.17.0 (marola-dev/marola-site#162) merged at 12:28:31Z, unattended.

    Named test: a skills-update run in a consumer after the bump. marola-site's skills.yml (skills-update.yml@v0.17.0, passes only pr_bot_key) dispatched at 12:32Z: run 39032904122, success. Steps: marola-pr-bot token success, both checkouts success, skills-vendor update --all … success with every skill is at its pin, so no PR was opened or refreshed. The workflow no longer declares token, and no caller passes it (marola, marola-site, the devkit's own skills.yml).

  7. self-assigned this
    on Oct 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

agent-readyPasses the five-rule Definition of Ready — an agent may begin work on it (MIP-0063 §5.4)area/dev-toolingClaude Code / OpenCode / agentic-tooling dev workflow, not the productenhancementNew feature or requestlayer/infra.github/, scripts/, justfile, flake.nix, .githooks/, .claude/size/SUnder ~100 changed lines

Type

No type

Projects

  • Status
    Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions