Repository navigation
template: checklist build output, index.js bundle, bin/ as build product - #40
Merged
Merged
Conversation
Build output (build/ui.mk) is now a checklist instead of cargo's verb column: one row per step with a green check, cyan tag and the artifact it produced; a spinner animates the row while the step runs (terminal, serial builds only), its output replayed above the finished row; a `done` row carries the total time; then each artifact on its own `==> path (size)` row. Commands are hidden, `make V=1` echoes them. Color only on a terminal with NO_COLOR honored; marks and spinner fall back to ASCII outside UTF-8. The bundle is written to dist/index.js: it is on yeet's entry ladder and main.jsx mounts its root explicitly, so the .jsx extension carried no meaning. bin/ is now purely a build product: the tracked .gitkeep is gone, /bin/ is ignored wholesale, `make clean` removes it and the `bin` rule recreates it. The kernel matrix stages veristat in .kmatrix/ (its own workspace) instead of bin/, locally and in CI; the in-VM gate's default path follows. scripts/new: no make directory banner, the next-step hint says `make` (there is no `yeet build`), rows styled like the build's, and the C locale is scoped to the name check so the marks match. toolchain.mk and fetch-toolchain.sh print in the same style; mirrored into toolchain/embed/ and still to be upstreamed to yeet-src/toolchain.
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
Checklist build output (
build/ui.mk), replacing the cargo-style verb column from #38/#39:-jkeeps static rows), with the command's output captured and replayed above the finished row, so a compiler error still lands right before its red✘ … (failed)row.donerow with the total build time; artifacts listed below as bold==> path (size), path magenta, size bold green.make V=1echoes each one before its row. Color only when stderr is a tty,TERM != dumb,NO_COLORunset. Marks and spinner fall back to ASCII outside a UTF-8 locale.make cleanprints one row per removed path.scripts/newprints in the same style, without make's directory banner, and its next-step hint saysmake(there is noyeet build). The C locale it forced is now scoped to the name check so the marks match.Bundle is
dist/index.js.index.jsis on yeet's entry ladder andsrc/main.jsxmounts its root explicitly, so the.jsxextension carried no meaning. Verified by running the app under a pty with both names.bin/is a build product.template/bin/.gitkeepremoved,/bin/ignored wholesale,make cleanremoves it, thebinrule recreates it. A fresh scaffold has nobin/until built, andgit statusis clean after a build-then-clean cycle.Kernel matrix stages veristat in
.kmatrix/(its own, gitignored workspace) instead ofbin/:build/kernel-matrix.sh(which now creates.kmatrix/before staging rather than wiping it afterwards),build/verify-kernel.sh's default path, and.github/workflows/kernel-matrix.yml.Test plan
V=1, deliberate compile error,make -j4(no spinner),LC_ALL=CASCII fallback.yeet run .starts the TUI fromdist/index.jsunder a pty.bin/before the first build; removed bymake clean; cleangit statusafter build + clean in a committed scaffold.scripts/newoutput on a tty and piped; invalid names still rejected.sh -non both kernel-matrix scripts; nobin/veristatreferences remain.make veristat-matrixand the CI kernel-matrix workflow (boot VMs; not run).