topdata: stop hardcoding the skills dataset location #43

Merged
archvillainette merged 1 commits from skills-dataset-tree into main 2026-07-19 07:33:20 +00:00
Owner

sow-topdata moved skills into a tree (skills/core, skills/specs, skillcats, skillspecranks), which broke validate/build: the generated feat families hardcoded dataset "skills".

Resolution is now data-driven: the family spec's child_source names the dataset, child slugs come from the row key's own namespace, and the wiki loader accepts both the flat and tree layouts (topdata main still uses the flat one until the overhaul merges).

Verified against the skill-grouping-overhaul topdata branch: validate/build/wiki all clean with zero lock churn, focus feats stay on their adopted rows.

🤖 Generated with Claude Code

sow-topdata moved skills into a tree (skills/core, skills/specs, skillcats, skillspecranks), which broke validate/build: the generated feat families hardcoded dataset "skills". Resolution is now data-driven: the family spec's child_source names the dataset, child slugs come from the row key's own namespace, and the wiki loader accepts both the flat and tree layouts (topdata main still uses the flat one until the overhaul merges). Verified against the skill-grouping-overhaul topdata branch: validate/build/wiki all clean with zero lock churn, focus feats stay on their adopted rows. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
archvillainette added 1 commit 2026-07-19 07:25:41 +00:00
topdata: stop hardcoding the skills dataset location
build-binaries / build-binaries (pull_request) Successful in 2m9s
test / test (pull_request) Successful in 1m23s
adffb1e11d
The skills dataset moved into a tree (skills/core, skills/specs, ...).
Resolve it data-driven instead of by name:

- The required-family check no longer pins skill focus families to
  dataset "skills"; the family spec's child_source names the dataset.
- Affinity generation and display names use the skill_focus spec's
  child_source dataset.
- Child slugs derive from the row key's own namespace (text after the
  first colon) instead of assuming namespace == dataset path.
- The wiki loader accepts both layouts ("skills", then "skills/core").

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
archvillainette merged commit bc8fa3a6fe into main 2026-07-19 07:33:20 +00:00
archvillainette deleted branch skills-dataset-tree 2026-07-19 07:33:20 +00:00
Sign in to join this conversation.