Skip to content

Fix validation errors introduced in #1127 - #1129

Open
GitWave22 wants to merge 1 commit into
opensource-observer:mainfrom
GitWave22:fix-1127-validation-errors
Open

Fix validation errors introduced in #1127#1129
GitWave22 wants to merge 1 commit into
opensource-observer:mainfrom
GitWave22:fix-1127-validation-errors

Conversation

@GitWave22

Copy link
Copy Markdown
Collaborator

main is currently failing CI. Three separate issues, all introduced in e5142b7 (#1127), block pnpm lint and pnpm validate for every PR opened since.

1. data/projects/n/netlink.yaml — filename does not match name

AssertionError: Filename must match name(netlinklabs): data/projects/n/netlink.yaml

Fixed by renaming the file to netlinklabs.yaml rather than changing the name field: netlinklabs matches the project's GitHub org (github.com/netlinklabs), which is the preferred naming pattern, and the slug is what downstream consumers key on. The project has no logo file, so nothing else needed moving.

2. data/projects/b/brix.yaml — duplicate GitHub URL

AssertionError: Duplicates found: {
  "https://github.com/inverternetwork": [
    "data/projects/i/inverternetwork.yaml",
    "data/projects/b/brix.yaml"
  ]
}

brix.yaml claimed https://github.com/inverternetwork, which belongs to inverternetwork.yaml. These are distinct projects — Brix brings emerging-market real-world yield onchain, Inverter is a token-economy protocol whose tech Brix appears to build on.

Brix has no GitHub org of its own that I could verify (brix.money links to none; github.com/brix is an unrelated project at brix.github.io), so the incorrect github field is removed rather than reassigned. github is optional in the schema. Happy to point it at the right org if a maintainer or the Brix team knows it.

3. data/projects/z/zora.yaml — Prettier formatting

Single-quoted strings where Prettier wants double quotes. Purely cosmetic — I verified the parsed YAML is identical before and after.

Verification

Run locally against this branch:

  • validate-projectsSuccess! Validated 7090 projects
  • validate-logosSuccess! Validated 4118 logo files
  • validate-collectionsSuccess! Validated 117 collections
  • tsc --noEmit — clean
  • prettier --check '**/*.{ts,yaml}'All matched files use Prettier code style!

No project data is changed beyond the three fixes above.

Found while preparing #1128; that PR is red for these same reasons and should go green once this merges.

🤖 Generated with Claude Code

Three issues left main failing CI:

- data/projects/n/netlink.yaml had name: netlinklabs, which did not match
  its filename. Renamed the file to netlinklabs.yaml, since netlinklabs
  matches the project's GitHub org and is the slug consumers key on.
- data/projects/b/brix.yaml claimed https://github.com/inverternetwork,
  which is already claimed by inverternetwork.yaml, tripping the duplicate
  check. Brix has no GitHub org of its own, so the field is removed.
- data/projects/z/zora.yaml used single-quoted strings where Prettier
  wants double quotes.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant