Skip to content

feat(game): stream validation for game metadata#119

Closed
Seungpyo1007 wants to merge 1 commit into
mainfrom
chore/game-data-validation
Closed

feat(game): stream validation for game metadata#119
Seungpyo1007 wants to merge 1 commit into
mainfrom
chore/game-data-validation

Conversation

@Seungpyo1007

Copy link
Copy Markdown
Member

What & why

Promotes games to a first-class validated data type without changing TechEngine.

  • validates game title, verified, rating/playtime counts, collection fields, ESRB values, image URLs, slug uniqueness, and deterministic shard paths;
  • adds python -m app.validate --category game, which walks the million-record game dataset without retaining JSON records in memory;
  • documents the game layout and sparse checkout workflow; and
  • adds focused unit coverage for valid records, invalid metadata, and shard placement.

Source

Not applicable: this is validator and repository-structure work. The fields are aligned with the existing TechEngine Game model and response schema; no engine code or submodule pointer is changed.

Checklist

  • python -m unittest tests.test_game_validation -v passes locally
  • Representative existing game record passes the new rules
  • Full python -m app.validate (all categories) is intentionally deferred to CI because this working copy uses sparse checkout
  • Full streaming game-dataset scan is in progress against the complete local clone; result will be added here
  • Game paths use data/game/<slug[:2]>/<slug>.json
  • Game slugs and source URL rules are validated

@github-actions github-actions Bot added app Validator or application code changes documentation Improvements or additions to documentation enhancement New feature or request labels Jul 13, 2026
@Seungpyo1007 Seungpyo1007 moved this from Todo to In Progress in TechAPI-Project Jul 13, 2026
@Seungpyo1007 Seungpyo1007 reopened this Jul 13, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Jul 13, 2026
@Seungpyo1007 Seungpyo1007 moved this from Done to In Progress in TechAPI-Project Jul 13, 2026
@Seungpyo1007

Copy link
Copy Markdown
Member Author

Superseded by #120 because GitHub did not refresh this PR's head SHA after the follow-up validation fix.

@github-project-automation github-project-automation Bot moved this from In Progress to Done in TechAPI-Project Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app Validator or application code changes documentation Improvements or additions to documentation enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants