Repository navigation
lorenzo/start kw integration pr 8 - #206
Draft
lorenzoberts wants to merge 6 commits into
Draft
lorenzoberts wants to merge 6 commits into
lorenzoberts wants to merge 6 commits into
Conversation
This commit adds target_kernel_tree to the config update pipeline so Edit Config can later select among existing kernel_trees keys without hand-editing JSON. An empty string unsets the target; an unknown key is refused and leaves the current value unchanged. Trees themselves are still added by editing the config file. This commit is part of the kw integration's step 8. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit adds a Target Kernel Tree row to Edit Config so the user can pick among existing kernel_trees keys, including <none> to unset. Cycling is a state-level operation for now; input wiring comes next. Keys are sorted so the cycle order does not depend on HashMap iteration. This commit is part of the kw integration's step 8. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit maps Left/Right while editing so the target kernel tree row can be cycled from the keyboard, shows <none> in the live edit cursor, and documents the interaction in help and the key hint. ESC/q already saved and exited; the help text now matches that. This commit completes the kw integration's step 8. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit treats a target_kernel_tree whose key is missing from kernel_trees as unset when opening Edit Config, so saving an unrelated field cannot abort the app. Trees are JSON-only, so that on-disk state is reachable. The repair is logged; surrounding whitespace is trimmed so it matches validation. Unknown-key errors now list known keys, and ESC/q is described as save-and-exit in the nav hint. This commit is part of the kw integration's step 8. Signed-off-by: lorenzoberts <lorenzobs@usp.br>
Signed-off-by: lorenzoberts <lorenzobs@usp.br>
This commit reports what kw deploy exits 0 through: initramfs and mkinitcpio failures, and a GRUB update that never listed the new kernel. These show as "succeeded ... with N warnings" on KwOps, and build records and deploy-alone are unchanged. A failed job shows the first error from its log, and every job shows the full log path. With a kw env active, an in-tree .config or include/config/ now makes the tree not ready, since kbuild refuses that O= build. A failed git am deletes the empty branch it created, and the popup names the branch the tree is back on. Signed-off-by: lorenzoberts <lorenzobs@usp.br> Co-authored-by: Cursor <cursoragent@cursor.com>
lorenzoberts
added this pull request to stack #200
October 4, 2026 18:56
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
1 participant
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.
Stack created with GitHub Stacks CLI • Give Feedback 💬