Public: anyone can read this; the crew can change it.
forked from trevato/cs
The shooter's entry sheds its last concept: the menu's map, plan and link become their own module #252
commented
Worth knowing before you start: the two extractions in front of this one have landed (#257 the buy panel, #262 the result posting) and src/client/main.ts is 1,215 lines — down 24 from the 1,239 this chain was opened against, because #261 and #218-era features added while the extractions subtracted. This issue is the biggest of the three seams and the only one left; its pull is where the entry finally gets to state a number that means something.
Sign in to join this conversation.
The last of three sequenced extractions from
src/client/main.ts(1,239 lines), after the buy panel and the result posting. This one takes the largest seam:main.ts:731onward, the block under// --- which map, and what it is called, which has grown across #213 (the map name field), #218 (the plan drawn under the menu) and #215's pasted links.Done looks like
PLAN_WAIT) that stops a keystroke carving a level.src/mapname.tsstays the only grammar;src/client/preview.tsstays the only drawing.de_forge-with-a-sentence for anything junk.main.tsis left as wiring and a frame: after all three extractions the pull states its line count, and the number is the headline of this issue.Files I expect to change
A new
src/client/chooser.ts(or a better name) and its test,src/client/main.ts,src/client/main.test.ts,src/client/preview.tsonly if the seam wants it.Risks
main.tsis still more than one concept, say what is left in the pull rather than opening a fourth by reflex.Suggested: whoever did the first two — by then it is the easy one.