docs: remove ntk tailwind, document the real dev loop - #37
Merged
Conversation
Released ntk has no tailwind subcommand and ntk watch does not compile Tailwind; both claims came from an unmerged theme-kit branch. Per the decision on NextCommerceCo/theme-kit#31 (ntk stays frontend-framework- agnostic), remove the ntk tailwind rows, correct the ntk watch description, and promote make dev from its Legacy label to the standard loop. Add make watch to the README table for the two-terminal setup. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
next-devin
marked this pull request as ready for review
July 23, 2026 09:18
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Reviewed by minimax-m3 · Input: 35.7K · Output: 2.3K · Cached: 246.3K |
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.
Problem
Spark's README and CLAUDE.md document
ntk tailwind/ntk tailwind --minifyand claimntk watchauto-compiles Tailwind. Neither is true of released ntk (1.1.1): the tailwind subcommand exists only on an unmerged theme-kit branch, andntk watchonly watches and pushes files (its compile step coverssass/, which Spark does not use). Anyone following the docs hits a hard failure at the first CSS change.Alex ruled on NextCommerceCo/theme-kit#31 that ntk stays frontend-framework-agnostic, so the fix is to remove the references rather than wait for a release.
What changed
ntk tailwindrows from the README command table and the CLAUDE.md dev block.ntk watchdescription in both files: it watches and pushes, and the Tailwind compile comes from the standalone binary.make dev(Tailwind watcher +ntk watchin parallel) from its Legacy label to the standard dev loop, which matches what the Makefile actually does. Addedmake watchto the README table for the two-terminal setup, and dropped the stale Legacy labels onmake css/make build.Verification
grep -rn 'ntk tailwind' README.md CLAUDE.md docs/is clean.python3 -m unittest discover -s testspasses.Part of the storefront theme Epic (NextCommerceCo/skills#29). theme-kit#31 can close once this lands; the remaining
ntk tailwindreferences in the Shopify guide are covered by NextCommerceCo/guides#38.🤖 Generated with Claude Code