sys.indicator, sys.video, and state_initials - #5
Merged
Merged
Conversation
IndicatorPlot draws several countries on one axis; sys.indicator answers each one's numbers beside it. Both name WHAT to plot and the runtime fetches it — a series written into a card is a fact (§4) and it is wrong the moment the World Bank revises it, which they do backwards, yearly. state_initials() is new and load-bearing: a host resolving a source argument BEFORE realize (which countries to ask about, before there is a tree) cannot read the store — nothing is written yet — or the seed data, because an initial is a declaration. Without it the request went out with an empty country list and the rows came back empty while the chart drew fine, which is exactly the accepted-card-wrong-screen class. Two more measured the same way. The text arguments needed Debug quoting: a bare interpolation emitted sys.indicator(CHN,IND, ...) — two arguments where one was meant, answering nothing. And a bare read.title now reads row 0 rather than nothing: the indicator's own name is the same for every row, and the card titles itself with it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
A YouTube search as a row source: id, title, channel, length, views, age, a thumbnail url and `embed` — a player url the HELPER builds, because L0 has no string concatenation and a card must not assemble a URL. Thumb is Photo's row-sized sibling: Photo fills its width because it is a backdrop, and a list row needs a fixed 16:9 tile beside its text. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
🤖 AI Code ReviewClaude CLI exited with code 1: ✅ No significant issues found. Tokens used: 0 | Powered by r9s.ai |
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.
Summary
Two capabilities, both following the rule the profile is built on — a card names WHAT it needs and the runtime fetches it, so no fact is ever written into a card.
sys.indicator+IndicatorPlot— compare countries on one World Bank series. The plot takes countries, an indicator code and a span; the source answers each country's numbers from the same request, indexed in the order the card listed them (so row 0's figures and the first line on the chart are the same country).sys.video+Thumb— a YouTube search as a row source: id, title, channel, length, views, age, thumbnail, andembed— a player url the helper builds, because L0 has no string concatenation and a card must not assemble a URL.ThumbisPhoto's row-sized sibling (Photo fills its width because it is a backdrop; a list row needs a fixed 16:9 tile).state_initials()is new and load-bearing: a host resolving a source argument BEFORE realize — which countries to ask about, before there is a tree — cannot read the store (nothing written yet) or the seed data (an initial is a declaration, not data). Without it the request went out with an empty country list and the rows came back empty while the chart drew fine: an accepted card with half a screen wrong.Also here, each found on a phone rather than in a test:
sys.indicator(CHN,IND, …), two arguments where one was meant, answering nothing;read.titlenow reads row 0, since the indicator's own name is the same for every row and the card titles itself with it;sys.link) is not reported dead.252 profile tests green.
🤖 Generated with Claude Code