rayfin up silently skips schema changes
Nobody has claimed this yet.
- Dominant language
- No language data
- Stars
- 610
- Forks
- 62
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 3
Description
Problem statement
rayfin up deploys code but does not apply entity schema changes. Those require the separate rayfin up db apply. Nothing warns you when the entity definitions have drifted from the deployed schema, so a deploy appears to succeed while the new entity does not exist, and the failure only shows up at runtime as a query error.
Evidence. Adding the AppAdmin entity required npx rayfin up db apply --yes before npx rayfin up --yes. Running only the latter produced a clean success with an app that could not read the new entity.
Impact. A silent, deferred failure. In CI this ships a broken build with a green light.
Proposed solution
Compare the local entity definitions against the deployed schema during rayfin up and either apply automatically, prompt, or fail with a clear instruction. At minimum, print a warning naming the drifted entities. A rayfin up --with-db convenience flag would also remove the ordering trap.
Alternatives considered
No response
Additional context
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the rayfin up and rayfin up db apply entry points, reproducing the AppAdmin example with the commands in the issue. Trace how local entity definitions and the deployed schema are handled; done should be a defined warning, prompt, automatic apply, or failure that prevents silent drift and names the affected entities.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- node.js
- Domain
- cli, databases
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 45/100