Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion .github/check-token-names.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,26 @@ def main() -> int:
value.upper(), []
).append(slug)

# §11.4.1 — every palette carries a reference name, and the TOML is where it
# lives. The Standard and the skills both print these names in prose, so a
# palette without one looks complete everywhere a reader looks and is empty
# in the only place a program reads. Three palettes shipped that way at
# v2.08 and nine gates let it through, because every other check asks about
# values and this is the one required key that is not a colour.
missing = sorted(s for s, p in palettes.items() if not p.get("reference"))
if missing:
for slug in missing:
print(f"`[palettes.{slug}]` has no `reference` key (§11.4.1).", file=sys.stderr)
print(
" Every palette carries a reference name for prose; the slug stays the\n"
" machine identifier. Add `reference = \"<name>-color-palette\"`.",
file=sys.stderr,
)
return 1

collisions = {name: v for name, v in seen.items() if len(v) > 1}
if not collisions:
print(f"token names OK ({len(seen)} distinct names across {len(palettes)} palettes)")
print(f"token names OK ({len(seen)} distinct names across {len(palettes)} palettes; all carry a reference)")
return 0

for name, by_hex in sorted(collisions.items()):
Expand Down
3 changes: 3 additions & 0 deletions steelbore-color-palette/assets/steelbore.toml
Original file line number Diff line number Diff line change
Expand Up @@ -800,6 +800,7 @@ border = "11.79:1"
# still keeps Hot Pink above 4.5:1 on it (4.72:1) — pinker fails the floor.
# -------------------------------------------------------------------------
[palettes.steelbore-blackpinkpanther]
reference = "blackpinkpanther-color-palette"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Rebuild the palette bundles with the reference keys

At this commit, both steelbore-color-palette.zip and steelbore-color-palette.skill are byte-identical to their parent versions, so their bundled assets/steelbore.toml does not contain the newly added Green/Green Alt reference keys and remains substantially behind the source file. Consumers installing either bundle therefore do not receive this fix; rebuild and commit both archives alongside the asset change.

AGENTS.md reference: AGENTS.md:L98-L101

Useful? React with 👍 / 👎.

"Runway Black" = "#000000"
"Backlit Rose" = "#2E0020"
"Backstage Black" = "#18000E"
Expand Down Expand Up @@ -897,6 +898,7 @@ border = "8.31:1"
# For a neutral-text reading of the same colours, see steelbore-greenalt.
# -------------------------------------------------------------------------
[palettes.steelbore-green]
reference = "green-color-palette"
"Vampire Black" = "#0D0208"
"Rain Shadow" = "#08180C"
"Royal Black" = "#030303"
Expand Down Expand Up @@ -992,6 +994,7 @@ border = "7.87:1"
# canvas, surfaces and status tones; a project picks one, never both (§11.4).
# -------------------------------------------------------------------------
[palettes.steelbore-greenalt]
reference = "greenalt-color-palette"
"Vampire Black" = "#0D0208"
"Rain Shadow" = "#08180C"
"Royal Black" = "#030303"
Expand Down
Loading