Repository navigation
feat(mapper): take .reylo.yaml as a content mapper extension - #11
Merged
Merged
Conversation
The setup now lists .reylo.yaml beside .sql, so the project file is Reylo's to type rather than a YAML loader's. Its module declares nothing, and a <name>.reylo.yaml that include takes in is no longer read as a query.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
This review covers the full PR: it maps .reylo.yaml through the content mapper beside .sql, and it updates the documented setup and every example project to match.
- Project file short-circuit in
src/mapper.ts:transformreturns an empty module (export {};) for any file ending inPROJECT_FILE. It does this before the project lookup, so neither the root.reylo.yamlnor a<name>.reylo.yamlthatincludetakes in is analyzed as SQL. - Setup docs:
README.mdandsite/content/sections/01-setup.mdx(including its agent prompt) now list[".sql", ".reylo.yaml"]. - Examples: all ten
examples/*/tsconfig.jsonfiles and their READMEs' "How Reylo is wired in" sections name the new extension. - New test
tests/project-file-module.test.ts: runstsc --runExternalCodeover a temporary SQLite project that imports.reylo.yamland includesconfig/staging.reylo.yaml. It uses the same scaffolding astests/definition.test.ts.
I ran the new test locally and it passes. With the endsWith(PROJECT_FILE) line removed, it fails with reylo6 syntax errors on both YAML files, so it covers the change. The only contentMappers configs still listing just .sql are under lab/ and in two existing tests, and none of those are user-facing.
claude-opus-5-5 ｜ 𝕏
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The setup now lists
.reylo.yamlbeside.sqlin the content mapper's extensions, in the README, the setup page (and its agent prompt) and all ten example projects.src/mapper.ts). Before, a mapped.reylo.yamlwas analyzed as SQL and failed with a syntax error; so did a<name>.reylo.yamlthatincludetakes in..yamlloader, since the longest registered suffix wins. A root.reylo.yamlstays out of the program unless imported, becauseincludewildcards skip dotfiles.tests/project-file-module.test.ts, which runstsc --runExternalCodeover a SQLite project that imports.reylo.yamland takes in aconfig/staging.reylo.yaml; it fails without the mapper change.Verified locally on
7.1.0-dev.20260918.1and7.1.0-beta: typecheck, loader tests, package build, docs site, and every example project's gate.