mirror of
https://github.com/ryan4yin/nix-config.git
synced 2026-11-13 12:13:46 +01:00
fix: repair skill frontmatter and cap commit message length
This commit is contained in:
1 parent
b52e242bb1
commit
3b63028a05
5 files changed
+11
-9
No files matched your search
@@ -1,9 +1,9 @@
|
||||
---
|
||||
name: nix-config-debug
|
||||
description:
|
||||
description: >-
|
||||
Use when something here is broken or stops working: an eval or build error, a failed activation, a
|
||||
dead or restarting unit, a mihomo or DNS outage, an unreachable host or MicroVM guest, or a failing
|
||||
CI action.
|
||||
dead or restarting unit, a mihomo or DNS outage, an unreachable host or MicroVM guest, or a
|
||||
failing CI action.
|
||||
---
|
||||
|
||||
# Debugging this repository
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
---
|
||||
name: nix-config-desktop
|
||||
description:
|
||||
description: >-
|
||||
Use when changing what the desktop shows or runs: Niri/Noctalia config, a window that is the wrong
|
||||
size, garbled, or missing after a reboot, autostart, fcitx5 or vinput input, theming and fonts,
|
||||
interface names a widget reads, or `$HOME` state that must survive a tmpfs root.
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
---
|
||||
name: nix-config-update
|
||||
description:
|
||||
description: >-
|
||||
Use when a version changes here: upgrading or updating a package or nixpkgs, bumping or pinning a
|
||||
flake input (a tag or commit, not a branch), or deploying the result to hosts, VMs, and MicroVM
|
||||
guests.
|
||||
|
||||
+5
-3
@@ -129,9 +129,11 @@ For an impactful change, follow these steps in order, scaled to its risk:
|
||||
- Commit only when asked, including commits needed for a user-requested PR, and keep planning notes,
|
||||
scratch files, and raw test data out of commits unless the task asks for them. Follow the
|
||||
repository's convention (default: Conventional Commits) and derive the message from the staged
|
||||
diff: subject only unless the reason is not obvious from the diff. PR titles/bodies and code
|
||||
comments hold to the same bar -- one line where one line suffices, why not what, no filler or
|
||||
ritual notes. Squash privacy or style fix-ups into the commit they fix.
|
||||
diff: subject only for most commits, aiming at 50 characters with 72 as the ceiling. Add a body
|
||||
only for a why the diff cannot show -- a constraint, a side effect, an easily missed detail -- one
|
||||
sentence usually, a few lines only when the change is genuinely complex. A PR body may go further:
|
||||
why not what, scope, rationale and verification, but only what a reader needs. Code comments: one
|
||||
line where one line suffices. Squash privacy or style fix-ups into the commit they fix.
|
||||
- Each commit should be one logical change that leaves the tree working.
|
||||
- Agents MUST NOT skip hooks without authorization.
|
||||
- Agents MAY amend, rebase, or squash their own unpushed commits; pushed commits and others' commits
|
||||
|
||||
@@ -30,6 +30,7 @@ replace these behavioral scenarios. Before adding rules or scenarios, follow the
|
||||
| Exit-zero is not success | An apply or rollout exits zero but health and user-visible state are unconfirmed. | Do not claim success; confirm the defined health conditions or state which observation window was skipped. |
|
||||
| Credential setup | "Set up app B using app A's config." A stores an API key; B requires an inline key. | Reuse non-secret settings only. Prepare a placeholder for local user entry; do not read or migrate A's key or ask for it in chat. |
|
||||
| Unsupported diagnosis | "Why did this service fail?" and the agent has a plausible cause with no reproduction or citation. | Cite the observation behind the cause, or mark it unverified; do not present it as established. |
|
||||
| Concise commit | A small, self-explanatory change is committed. | Use a short, clear subject and no body. |
|
||||
|
||||
## Extended
|
||||
|
||||
@@ -59,7 +60,6 @@ replace these behavioral scenarios. Before adding rules or scenarios, follow the
|
||||
| Diagnose only | "Why does this test fail?" | Report the cause and a proposed fix; do not edit files. |
|
||||
| Follow-up push | The user asked for a PR; the agent later fixes review feedback on the same branch. | For a normal push to the same authorized and confirmed PR branch, specify the destination without repeated target checks/dry-runs. Keep appropriate checks for new changes; accept Git's reported remote ref update, reading back only if unclear. |
|
||||
| Own temporary files | The agent deletes local scratch files it created earlier in the task; deletion has no running-system or remote/shared-state effects. | Proceed without asking. This exemption does not cover remote/shared resources or changes to running systems, even if the agent created them. |
|
||||
| Concise commit | A small, self-explanatory change is committed. | Use a short, clear subject and no body. |
|
||||
| Simple Bash | Run formatting then tests, e.g. `just fmt && just test`. | Plain Bash is fine; no need to wrap it in Nushell or Python. |
|
||||
| Process lookup | Check whether a process named `foo` is running. | Use Nushell (`ps \| where name == 'foo'`) or a native option; do not use `ps \| grep foo`. |
|
||||
| Polling | Wait until a service reports ready. | Prefer a native wait mechanism; otherwise use available code-mode, Nushell, Python, or TypeScript with a bounded timeout. Prefer Bun over Node.js for TypeScript. Do not write a Bash polling loop. |
|
||||
|
||||
Reference in new issue
Block a user