Mr Folio's /roadmap shows the releases still being built - #250
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cc956476bd
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| return { | ||
| name: section.release ? `Folio ${section.release}${section.subtitle ? ` (${section.subtitle})` : ''}` : section.title, | ||
| release: Boolean(section.release), | ||
| shipped: Boolean(section.release) && items.length > 0 && items.every((item) => item.status === 'done'), |
There was a problem hiding this comment.
Reserve space for every requested roadmap section
With the reviewed app/src/main/assets/roadmap.json, this retains four unfinished release sections in addition to Later and Exploring. The unchanged handler emits up to six items per section and only trims the fully assembled message afterward, so /roadmap reaches 1,999 characters partway through Folio 0.7.1, omits Later and Exploring entirely, cuts a sentence mid-word, and drops the whole-roadmap link; /roadmap next is truncated similarly. Budget or summarize each block before assembling the response so every requested section and the link remain visible.
Useful? React with 👍 / 👎.
What this changes
Mr Folio's
/roadmapis wrong on Discord right now, and it is my doing. The command treated every roadmap section with a version as "shipped" and left it out. The roadmap rewrite (#240, merged earlier today) moved "Next" into release sections (0.6.8, 0.6.9, 0.7.0, 0.7.1), so since then:/roadmaplists only Later and Exploring, and/roadmap nextanswers "Nothing under next."Now:
/roadmap nextmeans those releases, named with their theme: "Folio 0.6.9 (Foundation)". Later and Exploring work as before.sources.mjspassesreleasealong, and reads the optionalsubtitlethe new roadmap page uses.How I checked it
node --testintools/folio-bot: 28 pass (one test changed to the new shape, one added fornext).roadmap.jsononmain:/roadmapand/roadmap nextshow Folio 0.6.8 and Folio 0.6.9 (Foundation) with their items, and/roadmap latershows Later.tools/folio-bot,~/dev/folio-site/node_modules/.bin/wrangler deploy(npx wrangler@4hangs, per the notes). Until then/roadmapstays wrong.Cheap way out: revert; no other command reads these fields.