Skip to content

slim and correct agent instructions, fold in AGENTS.override.md - #1

Open
nikitacometa wants to merge 1 commit into
mainfrom
docs/agent-instructions-20260928
Open

nikitacometa wants to merge 1 commit into
mainfrom
docs/agent-instructions-20260928

Conversation

@nikitacometa

Copy link
Copy Markdown
Owner

CLAUDE.md (460 строк, 35 КБ) и AGENTS.md не менялись с начала июня, а репозиторий с тех пор вышел в mainnet. Оба файла называли живым VRF beacon 947957720, который код в трёх местах называет мёртвым с 2022 года, расходились с кодом box MBR, house edge и referral bps, домен (.xyz в доках, .quest в проде). Codex вместо AGENTS.md читал локальный незакоммиченный AGENTS.override.md. Команды компиляции и генерации клиентов в доках ссылались на несуществующий packages/contracts/artifacts/.

• AGENTS.md (249 строк): общий файл для Claude, Codex и Cursor, полезное из AGENTS.override.md (маршрутизация задач, сводка money-path) перенесено сюда. Раздел «Stale facts» перечисляет устаревшие доки, которые этот PR не правит: beacon ID в docs/tokenomics.md и skill-референсах, расхождения house economics, домен.
• CLAUDE.md: @AGENTS.md, позиционирование, таблица MCP, триггеры skills и что сохранять при compact.
• .claude/rules/contracts-vrf-treasury.md: money-path инварианты контрактов, VRF и treasury. Claude грузит их для packages/contracts/**, packages/sdk/src/vrf/** и packages/sdk/src/clients/**, для Codex то же лежит в «Critical invariants» AGENTS.md.
• Компиляция: algokit compile python smart_contracts/<contract>/contract.py --out-dir artifacts для каждого из четырёх контрактов. Без --out-dir puyapy 5.8.1 пишет рядом с contract.py (puyapy/compile.py:128-129 в puya@v5.8.1), и make contracts-generate читает старые ABI. Исправлено в AGENTS.md, README.md, packages/contracts/README.md (добавлен пропущенный fairjackpot), packages/sdk/src/clients/README.md, docs/architecture.md и skill fairground-contract.
• Генерация клиентов: везде make contracts-generate. Пути к артефактам исправлены на smart_contracts/<contract>/artifacts/ в тех же файлах, в docs/game-specs/coinflip.md и в комментариях Makefile и packages/sdk/src/index.ts. Убрано заявление, что pnpm contracts:* и make contracts-build/test работают: они вызывают algokit project run, а [project.run.*] в .algokit.toml не настроен.
• Совместимость ARC-56: generator 6.0.1 принимает вывод puyapy 5.8.1 без ARC-32 fallback. Статус «resolved» возвращён в комментарий packages/contracts/.algokit.toml, packages/contracts/README.md и docs/architecture.md.
• Порядок удаления box в skills puya-gotchas и fairground-contract приведён к coinflip/contract.py: resolve() платит выигрыш, удаляет box, потом возвращает MBR (строки 260-265), refund() удаляет box и одним платежом возвращает bet + MBR (285-286). Старое правило «box всегда удаляется последним» ломало бы refund.
• Таблицы в README.md, docs/architecture.md и skill fairground-contract выровнены prettier из pre-commit хука.

Исполняемые target'ы Makefile, скрипты package.json и код контрактов не менялись.

Проверки:
• pnpm turbo typecheck и pnpm turbo lint: 16/16, exit 0.
• npx eslint packages/sdk/src/index.ts: exit 0.
• npx prettier --check по изменённым markdown-файлам: exit 0. Husky pre-commit (lint-staged) прошёл.
• git check-ignore -v --no-index для artifacts/*.arc56.json и пути рядом с contract.py: оба exit 1, не игнорируются.
• Поведение puyapy без --out-dir взято из исходника компилятора v5.8.1. Компиляция не запускалась: puyapy в окружении не установлен.

Вопросы:
• Чинить make contracts-build/test и pnpm contracts:* отдельной задачей: добавить [project.run.*] в .algokit.toml или перейти на прямые команды?
• Секция «Contract Client (TODO)» в apps/game/README.md устарела целиком: ссылается на нерабочие pnpm contracts:* и пишет, что клиента нет, хотя packages/sdk/src/clients/CoinflipClient.ts есть. Убрать её?
• «Stale facts» в AGENTS.md оставляет открытыми: живой beacon ID для mainnet и TestNet, домен в коде против прода, geo-block, house economics bps, on-chain min/max bet, актуальность docs/roadmap.md и старых аудитов. Закрываем отдельным PR по докам?

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