epic: ai-skills to share with team#14
Conversation
…pection without access to the contract source code
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f913634. Configure here.
| atomic_write() { | ||
| local -r target="$1" | ||
| local -r tmpfile | ||
| tmpfile=$(mktemp) || return 1 |
There was a problem hiding this comment.
Readonly variable reassignment causes runtime error in example
Medium Severity
In the atomic_write function, local -r tmpfile declares tmpfile as readonly with an empty value. The immediately following tmpfile=$(mktemp) will fail at runtime with "bash: tmpfile: readonly variable" because a readonly variable cannot be reassigned. The declaration and assignment need to happen in a single statement (local -r tmpfile=$(mktemp)) or the -r flag needs to be dropped from the local declaration. Since this is a teaching document for defensive bash patterns, an AI agent following this pattern will produce broken code.
Reviewed by Cursor Bugbot for commit f913634. Configure here.


Note
Low Risk
Primarily adds and tweaks markdown documentation under
ai-skills/plus a README update; no production code paths or runtime behavior changes.Overview
Adds an
ai-skills/section to the rootREADME.mdand documents it as a shared library of Cursor/LLM operational playbooks.Introduces several new skills (e.g.,
bash-defensive-patterns,just,find-skills,drill-spec,lang-caveman,new-chat-prompt) and expands existing ones by tighteningcommit-diffcommit-message conventions, fixing theepics-plusexample flag, and adding anibidWasm query introspection tip.Includes minor doc formatting/clarity fixes in
sai-db/schema.md.Reviewed by Cursor Bugbot for commit f913634. Configure here.