handoff: ramble phase 2 shipped, heart containers live on grackle - #340
Merged
Conversation
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.
Handoff doc for the reward-economy phase 2 arc, shipped in #339 (
4a5c3215, ramble 0.10.0) and deployed to grackle.Docs-only. Records what shipped, the deploy verification, what is open for Kevin, the follow-ups, and three lessons that cost real rework:
SYNCED_TABLESthat is false — it is only ever a.includes()allowlist, and_applyEntryapplies one entry at a time in arrival order. The design was right, the reason was not, and it had shipped in a code comment where a future reader would check it, find it false, and revert a clamp that prevents permanent energy loss.Also records two operational facts worth not rediscovering: grackle's crow DB is
~/crow/data/crow.db, not~/.crow/crow.db(which is 0 bytes), while bundles still install under~/.crow/bundles/; and there is no CI gate for guide-doc i18n parity, contrary to what the plan claimed.