Skip to content

epic: ai-skills to share with team#14

Merged
Unique-Divine merged 11 commits into
mainfrom
ud/dev
Apr 28, 2026
Merged

epic: ai-skills to share with team#14
Unique-Divine merged 11 commits into
mainfrom
ud/dev

Conversation

@Unique-Divine
Copy link
Copy Markdown
Owner

@Unique-Divine Unique-Divine commented Apr 28, 2026

  • docs(README): Describe ai-skills directory
  • ai-skills(commit-diff): Take a more opinionated approach to the messages
  • ai-skills(epics-plus): Fix bug in example command
  • ai-skills: Upload justfile skill
  • ai-skills(sai-db): Correction in schema.md
  • ai-skills(nibiru-cli-nibid): Add note on how to query Wasm via introspection without access to the contract source code
  • ai-skills: Impl drill-spec
  • ai-skills: Impl lang-caveman
  • ai-skills: Impl bash-defensive-patterns
  • ai-skills: find-skills
  • ai-skills: new-chat-prompt

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 root README.md and 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 tightening commit-diff commit-message conventions, fixing the epics-plus example flag, and adding a nibid Wasm query introspection tip.

Includes minor doc formatting/clarity fixes in sai-db/schema.md.

Reviewed by Cursor Bugbot for commit f913634. Configure here.

@Unique-Divine Unique-Divine merged commit 12c142b into main Apr 28, 2026
3 checks passed
@Unique-Divine Unique-Divine deleted the ud/dev branch April 28, 2026 18:05
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f913634. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant