@mrciphersmith/keryx 0.2.9

package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.opencode.md

@@ -0,0 +1,204 @@
+---
+name: code-ai-review
+description: "Performs strict AI code review following code-review-ai-assistant.mdc standards. Reviews current branch changes from merge-base by default, including both committed and local uncommitted changes. Use when: code review requested, checking branch changes, reviewing implementation quality."
+triggers:
+  - "Code review"
+  - "Review my changes"
+  - "Check this code"
+  - "Review code"
+metadata:
+  author: "MrCipherSmith"
+  version: "1.0.0"
+  category: "review"
+license: "MIT"
+compatibility: "cursor,codex,zed,opencode"
+---
+
+
+# Code AI Review (только текущая ветка)
+
+## Workflow
+
+Copy this checklist and track progress:
+
+```
+Code Review Progress:
+- [ ] Step 1: Determine parent branch and calculate merge-base
+- [ ] Step 2: Collect git diff (committed + local changes)
+- [ ] Step 3: Identify changed files and categorize
+- [ ] Step 4: Review each file following standards
+- [ ] Step 5: Document findings with severity and location
+- [ ] Step 6: Generate report with patches
+```
+
+## Главное правило: скоуп ревью
+
+Ревьюй только изменения, внесённые в текущей ветке с момента её ответвления от родительской.
+
+- Если пользователь **не передал commit hash/range**, включай **весь срез ветки от merge-base до рабочего дерева**:
+  - закоммиченные (`BASE_SHA..HEAD`)
+  - локальные незакоммиченные (`staged/unstaged/untracked`)
+- Если пользователь **явно передал commit hash/range**, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
+- Не ревьюй не связанные с веткой части репозитория.
+
+## Определение parent ref (детерминированно, именно родитель ветки)
+
+Используй первый существующий вариант:
+
+1. `origin/main`
+2. `origin/master`
+3. `main`
+4. `master`
+5. `@{upstream}` только если это не текущая feature-ветка
+
+```bash
+BRANCH="$(git rev-parse --abbrev-ref HEAD)"
+UPSTREAM_REF="$(git rev-parse --abbrev-ref --symbolic-full-name @{upstream} 2>/dev/null || true)"
+
+PARENT=""
+if git rev-parse --verify -q "origin/main" >/dev/null; then
+  PARENT="origin/main"
+elif git rev-parse --verify -q "origin/master" >/dev/null; then
+  PARENT="origin/master"
+elif git rev-parse --verify -q "main" >/dev/null; then
+  PARENT="main"
+elif git rev-parse --verify -q "master" >/dev/null; then
+  PARENT="master"
+elif [ -n "$UPSTREAM_REF" ] && [ "$UPSTREAM_REF" != "$BRANCH" ] && [ "$UPSTREAM_REF" != "origin/$BRANCH" ]; then
+  PARENT="@{upstream}"
+else
+  echo "Cannot determine parent ref" >&2
+  exit 1
+fi
+
+BASE_SHA="$(git merge-base HEAD "$PARENT")"
+```
+
+## Команды, чтобы собрать “срез” для ревью
+
+### A) Режим по умолчанию (пользователь НЕ дал hash/range)
+
+```bash
+git status
+
+# Закоммиченная часть ветки
+git log --oneline "${BASE_SHA}..HEAD"
+git diff --stat "${BASE_SHA}..HEAD"
+git diff --name-status "${BASE_SHA}..HEAD"
+git diff "${BASE_SHA}..HEAD"
+
+# Полный текущий срез от merge-base до рабочего дерева:
+# включает commit'ы + staged + unstaged
+# (untracked смотри через git status / git ls-files)
+git diff --stat "${BASE_SHA}"
+git diff --name-status "${BASE_SHA}"
+git diff "${BASE_SHA}"
+git ls-files --others --exclude-standard
+```
+
+### B) Режим с явным hash/range
+
+```bash
+# Примеры:
+git show --stat --name-status --patch <COMMIT_SHA>
+git log --oneline <FROM_SHA>..<TO_SHA>
+git diff --stat <FROM_SHA>..<TO_SHA>
+git diff --name-status <FROM_SHA>..<TO_SHA>
+git diff <FROM_SHA>..<TO_SHA>
+```
+
+## Правила ревью (обязательно)
+
+Ты обязан следовать стандарту ревью из `.metaproject/rules/core/code-review-ai-assistant.mdc`:
+
+- Structure output as: short summary -> structured findings by category -> concrete suggestions / optional patches.
+- Prioritize correctness and safety over style.
+- Avoid noise; focus on actionable, high-signal findings.
+- Do not request large refactors unless explicitly asked.
+
+## Формат вывода (шаблон подробного отчёта)
+
+Пиши ревью по структуре ниже:
+
+```markdown
+## Краткое резюме
+<1-3 предложения: что сделано и общий вердикт (OK / needs work).>
+
+## Скоуп ревью (только текущая ветка)
+- Ветка: `<BRANCH>`
+- Parent ref: `<PARENT>`
+- Merge-base: `<BASE_SHA>`
+- Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
+- Коммиты (merge-base..HEAD): <N>
+- Изменённые файлы: <список или количество>
+
+## Находки
+### Correctness
+<находки>
+
+### Types & Safety
+<находки>
+
+### Architecture & State
+<находки>
+
+### Readability & Style
+<находки>
+
+### Performance
+<находки>
+
+### UX/UI
+<находки>
+
+### A11y
+<находки>
+
+### Tests
+<находки>
+
+## Предложенные исправления (patches)
+<минимальные, точечные диффы для самых очевидных фиксов>
+```
+
+### Формат каждой находки (обязательно)
+
+Для каждой находки укажи:
+
+- **Severity**: `blocker` / `major` / `minor`
+- **Location**: file path + relevant lines/hunk (from the diff)
+- **Problem**: what is wrong
+- **Why it matters**: correctness/safety/perf/maintainability impact
+- **Suggested fix**: concrete change
+- **Optional patch**: provide a minimal unified diff when straightforward
+
+Пример блока патча:
+
+```diff
+diff --git a/path/file.ts b/path/file.ts
+index 0000000..1111111 100644
+--- a/path/file.ts
++++ b/path/file.ts
+@@ -1,3 +1,3 @@
+-const x = 1;
++const x = 2;
+```
+
+---
+
+## Job Context Awareness
+
+When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
+
+```
+JOB_NAME:     <job-name>
+CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
+```
+
+If provided and the file exists, read the context document before starting the review. Use it to:
+- Understand which libraries and patterns were intentionally chosen for the implementation
+- Avoid flagging correct library usage as issues (e.g., if context documents the API pattern)
+- Provide more accurate findings by understanding the project's architectural decisions
+- Reference context when justifying suggestions
+
+If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.

package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.zed.md

@@ -0,0 +1,204 @@
+---
+name: code-ai-review
+description: "Performs strict AI code review following code-review-ai-assistant.mdc standards. Reviews current branch changes from merge-base by default, including both committed and local uncommitted changes. Use when: code review requested, checking branch changes, reviewing implementation quality."
+triggers:
+  - "Code review"
+  - "Review my changes"
+  - "Check this code"
+  - "Review code"
+metadata:
+  author: "MrCipherSmith"
+  version: "1.0.0"
+  category: "review"
+license: "MIT"
+compatibility: "cursor,codex,zed,opencode"
+---
+
+
+# Code AI Review (только текущая ветка)
+
+## Workflow
+
+Copy this checklist and track progress:
+
+```
+Code Review Progress:
+- [ ] Step 1: Determine parent branch and calculate merge-base
+- [ ] Step 2: Collect git diff (committed + local changes)
+- [ ] Step 3: Identify changed files and categorize
+- [ ] Step 4: Review each file following standards
+- [ ] Step 5: Document findings with severity and location
+- [ ] Step 6: Generate report with patches
+```
+
+## Главное правило: скоуп ревью
+
+Ревьюй только изменения, внесённые в текущей ветке с момента её ответвления от родительской.
+
+- Если пользователь **не передал commit hash/range**, включай **весь срез ветки от merge-base до рабочего дерева**:
+  - закоммиченные (`BASE_SHA..HEAD`)
+  - локальные незакоммиченные (`staged/unstaged/untracked`)
+- Если пользователь **явно передал commit hash/range**, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
+- Не ревьюй не связанные с веткой части репозитория.
+
+## Определение parent ref (детерминированно, именно родитель ветки)
+
+Используй первый существующий вариант:
+
+1. `origin/main`
+2. `origin/master`
+3. `main`
+4. `master`
+5. `@{upstream}` только если это не текущая feature-ветка
+
+```bash
+BRANCH="$(git rev-parse --abbrev-ref HEAD)"
+UPSTREAM_REF="$(git rev-parse --abbrev-ref --symbolic-full-name @{upstream} 2>/dev/null || true)"
+
+PARENT=""
+if git rev-parse --verify -q "origin/main" >/dev/null; then
+  PARENT="origin/main"
+elif git rev-parse --verify -q "origin/master" >/dev/null; then
+  PARENT="origin/master"
+elif git rev-parse --verify -q "main" >/dev/null; then
+  PARENT="main"
+elif git rev-parse --verify -q "master" >/dev/null; then
+  PARENT="master"
+elif [ -n "$UPSTREAM_REF" ] && [ "$UPSTREAM_REF" != "$BRANCH" ] && [ "$UPSTREAM_REF" != "origin/$BRANCH" ]; then
+  PARENT="@{upstream}"
+else
+  echo "Cannot determine parent ref" >&2
+  exit 1
+fi
+
+BASE_SHA="$(git merge-base HEAD "$PARENT")"
+```
+
+## Команды, чтобы собрать “срез” для ревью
+
+### A) Режим по умолчанию (пользователь НЕ дал hash/range)
+
+```bash
+git status
+
+# Закоммиченная часть ветки
+git log --oneline "${BASE_SHA}..HEAD"
+git diff --stat "${BASE_SHA}..HEAD"
+git diff --name-status "${BASE_SHA}..HEAD"
+git diff "${BASE_SHA}..HEAD"
+
+# Полный текущий срез от merge-base до рабочего дерева:
+# включает commit'ы + staged + unstaged
+# (untracked смотри через git status / git ls-files)
+git diff --stat "${BASE_SHA}"
+git diff --name-status "${BASE_SHA}"
+git diff "${BASE_SHA}"
+git ls-files --others --exclude-standard
+```
+
+### B) Режим с явным hash/range
+
+```bash
+# Примеры:
+git show --stat --name-status --patch <COMMIT_SHA>
+git log --oneline <FROM_SHA>..<TO_SHA>
+git diff --stat <FROM_SHA>..<TO_SHA>
+git diff --name-status <FROM_SHA>..<TO_SHA>
+git diff <FROM_SHA>..<TO_SHA>
+```
+
+## Правила ревью (обязательно)
+
+Ты обязан следовать стандарту ревью из `.metaproject/rules/core/code-review-ai-assistant.mdc`:
+
+- Structure output as: short summary -> structured findings by category -> concrete suggestions / optional patches.
+- Prioritize correctness and safety over style.
+- Avoid noise; focus on actionable, high-signal findings.
+- Do not request large refactors unless explicitly asked.
+
+## Формат вывода (шаблон подробного отчёта)
+
+Пиши ревью по структуре ниже:
+
+```markdown
+## Краткое резюме
+<1-3 предложения: что сделано и общий вердикт (OK / needs work).>
+
+## Скоуп ревью (только текущая ветка)
+- Ветка: `<BRANCH>`
+- Parent ref: `<PARENT>`
+- Merge-base: `<BASE_SHA>`
+- Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
+- Коммиты (merge-base..HEAD): <N>
+- Изменённые файлы: <список или количество>
+
+## Находки
+### Correctness
+<находки>
+
+### Types & Safety
+<находки>
+
+### Architecture & State
+<находки>
+
+### Readability & Style
+<находки>
+
+### Performance
+<находки>
+
+### UX/UI
+<находки>
+
+### A11y
+<находки>
+
+### Tests
+<находки>
+
+## Предложенные исправления (patches)
+<минимальные, точечные диффы для самых очевидных фиксов>
+```
+
+### Формат каждой находки (обязательно)
+
+Для каждой находки укажи:
+
+- **Severity**: `blocker` / `major` / `minor`
+- **Location**: file path + relevant lines/hunk (from the diff)
+- **Problem**: what is wrong
+- **Why it matters**: correctness/safety/perf/maintainability impact
+- **Suggested fix**: concrete change
+- **Optional patch**: provide a minimal unified diff when straightforward
+
+Пример блока патча:
+
+```diff
+diff --git a/path/file.ts b/path/file.ts
+index 0000000..1111111 100644
+--- a/path/file.ts
++++ b/path/file.ts
+@@ -1,3 +1,3 @@
+-const x = 1;
++const x = 2;
+```
+
+---
+
+## Job Context Awareness
+
+When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
+
+```
+JOB_NAME:     <job-name>
+CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
+```
+
+If provided and the file exists, read the context document before starting the review. Use it to:
+- Understand which libraries and patterns were intentionally chosen for the implementation
+- Avoid flagging correct library usage as issues (e.g., if context documents the API pattern)
+- Provide more accurate findings by understanding the project's architectural decisions
+- Reference context when justifying suggestions
+
+If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.

package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.codex.md

@@ -0,0 +1,209 @@
+---
+name: code-b091-review
+description: "Performs strict b091-style code review following code-review-b091-profile.mdc. Reviews current branch changes from merge-base. Direct, no-fluff feedback focused on logic correctness. Use when: b091 review requested, strict validation needed."
+triggers:
+  - "Review as b091"
+  - "b091 style review"
+  - "b091 review"
+metadata:
+  author: "MrCipherSmith"
+  version: "1.0.0"
+  category: "review"
+license: "MIT"
+compatibility: "cursor,codex,zed,opencode"
+---
+
+
+# Code Review as b091 (только текущая ветка)
+
+## Workflow
+
+Copy this checklist and track progress:
+
+```
+b091 Review Progress:
+- [ ] Step 1: Determine parent branch and calculate merge-base
+- [ ] Step 2: Collect git diff (committed + local changes)
+- [ ] Step 3: Apply b091 principles (logic in correct layer, no ducttape)
+- [ ] Step 4: Check types (no any, typed mocks)
+- [ ] Step 5: Verify conventions (currentState, I prefix, etc.)
+- [ ] Step 6: Challenge all assumptions
+- [ ] Step 7: Generate direct report with patches
+```
+
+## Обязательные правила
+
+1) **Скоуп по умолчанию (без commit hash/range в запросе)**: ревью включает **все** изменения в текущей ветке от точки ответвления (merge-base) от родительской ветки:
+   - закоммиченные (`BASE_SHA..HEAD`)
+   - локальные незакоммиченные (staged/unstaged/untracked)
+2) **Скоуп при явном commit hash/range**: если пользователь явно передал hash или диапазон, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
+3) **Стиль/принципы b091**: ревью строго по `.metaproject/rules/core/code-review-b091-profile.mdc` (разделы правил и промпт).
+4) **Результат**: подробный отчёт по проблемам с объяснением и предложением исправлений (минимальные патчи там, где это просто).
+
+## Определение parent ref (детерминированно, именно родитель ветки)
+
+Используй первый существующий вариант:
+
+1. `origin/main`
+2. `origin/master`
+3. `main`
+4. `master`
+5. `@{upstream}` **только если это не текущая feature-ветка**
+
+```bash
+BRANCH="$(git rev-parse --abbrev-ref HEAD)"
+UPSTREAM_REF="$(git rev-parse --abbrev-ref --symbolic-full-name @{upstream} 2>/dev/null || true)"
+
+PARENT=""
+if git rev-parse --verify -q "origin/main" >/dev/null; then
+  PARENT="origin/main"
+elif git rev-parse --verify -q "origin/master" >/dev/null; then
+  PARENT="origin/master"
+elif git rev-parse --verify -q "main" >/dev/null; then
+  PARENT="main"
+elif git rev-parse --verify -q "master" >/dev/null; then
+  PARENT="master"
+elif [ -n "$UPSTREAM_REF" ] && [ "$UPSTREAM_REF" != "$BRANCH" ] && [ "$UPSTREAM_REF" != "origin/$BRANCH" ]; then
+  PARENT="@{upstream}"
+else
+  echo "Cannot determine parent ref" >&2
+  exit 1
+fi
+
+BASE_SHA="$(git merge-base HEAD "$PARENT")"
+```
+
+## Команды, чтобы собрать “срез” для ревью
+
+### A) Режим по умолчанию (пользователь НЕ дал hash/range)
+
+```bash
+git status
+
+# Закоммиченная часть ветки
+git log --oneline "${BASE_SHA}..HEAD"
+git diff --stat "${BASE_SHA}..HEAD"
+git diff --name-status "${BASE_SHA}..HEAD"
+git diff "${BASE_SHA}..HEAD"
+
+# Полный текущий срез от merge-base до рабочего дерева:
+# включает commit'ы + staged + unstaged
+# (untracked смотри через git status / git ls-files)
+git diff --stat "${BASE_SHA}"
+git diff --name-status "${BASE_SHA}"
+git diff "${BASE_SHA}"
+git ls-files --others --exclude-standard
+```
+
+### B) Режим с явным hash/range
+
+Если пользователь дал конкретный hash/range, используй его буквально:
+
+```bash
+# Примеры:
+git show --stat --name-status --patch <COMMIT_SHA>
+git log --oneline <FROM_SHA>..<TO_SHA>
+git diff --stat <FROM_SHA>..<TO_SHA>
+git diff --name-status <FROM_SHA>..<TO_SHA>
+git diff <FROM_SHA>..<TO_SHA>
+```
+
+## Как ревьюить (по b091)
+
+Применяй чек-лист и промпт из `.metaproject/rules/core/code-review-b091-profile.mdc`. Ключевые акценты b091:
+
+- Логика должна жить в правильном слое (часто — в store), чтобы её можно было нормально тестировать.
+- Против “patch/glue/ducttape”: требуй чинить причину, а не маскировать симптомы.
+- В новом коде никаких `any`, `as any`, небезопасных кастов; моки — типизированные и не наследуются от реальных реализаций.
+- Не плодить дублирование правил/проверок: один источник правды, переиспользовать сеттеры/методы.
+- Конвенции проекта обязательны (например, `currentState`, `I` для интерфейсов, консистентные типы).
+- Магические числа / `setTimeout(0)` / "страшные" хаки — только с чётким "почему", либо перепроектировать.
+- **Accessibility модификаторы в сторах**: inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный метод допустим только если он вызывается из React-компонента. Спросить: "Вызывается ли этот метод из JSX?" Если нет — `private`.
+- С MobX по умолчанию **не добавлять** `useCallback`/`useMemo` без явной необходимости.
+- Предложения должны проходить линт; не нитпикать автоформатирование.
+- Не раздувать scope: большие рефакторы “not today”, если вне задачи.
+
+## Формат вывода (подробный отчёт + тон b091)
+
+Пиши по-русски, но допускай короткие b091-маркеры (например, `not today ;P`, “ducttape”, “broken thinking”) только по делу и без перехода на личности.
+
+Используй структуру:
+
+```markdown
+## Вердикт (b091)
+<OK / needs work / fundamentally wrong> + 1–3 самых важных тезиса.
+
+## Скоуп ревью (только текущая ветка)
+- Ветка: `<BRANCH>`
+- Parent ref: `<PARENT>`
+- Merge-base: `<BASE_SHA>`
+- Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
+- Коммиты (merge-base..HEAD): <N>
+- Изменённые файлы: <список или количество>
+
+## Находки (подробно)
+### Архитектура и место логики
+<находки>
+
+### Типы, касты, моки
+<находки>
+
+### Корректность и семантика состояний
+<находки>
+
+### Конвенции и консистентность
+<находки>
+
+### Линт/формат/шум
+<находки>
+
+### UX/edge cases
+<находки>
+
+### Тестируемость
+<находки>
+
+## Предложенные исправления (patches)
+<минимальные unified diff-патчи для очевидных правок>
+```
+
+### Формат каждой находки (обязательно)
+
+Для каждой находки укажи:
+
+- **Severity**: `blocker` / `major` / `minor`
+- **Location**: путь файла + релевантный hunk/фрагмент из diff
+- **Problem**: что не так
+- **Why it matters**: почему это важно (корректность/тестируемость/конвенции/поддерживаемость)
+- **Suggested fix**: что конкретно сделать (точечно, без раздувания scope)
+- **Optional patch**: если фикс простой — unified diff
+
+Пример блока патча:
+
+```diff
+diff --git a/path/file.ts b/path/file.ts
+index 0000000..1111111 100644
+--- a/path/file.ts
++++ b/path/file.ts
+@@ -1,3 +1,3 @@
+-const x = 1;
++const x = 2;
+```
+
+---
+
+## Job Context Awareness
+
+When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
+
+```
+JOB_NAME:     <job-name>
+CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
+```
+
+If provided and the file exists, read the context document before starting the review. Use it to:
+- Understand which libraries and patterns were intentionally chosen for the implementation
+- Avoid flagging correct library usage as issues
+- Provide more accurate findings by understanding the project's architectural decisions
+
+If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.