Repository navigation
Разрезать модуль стройки construction.ts на части - #2107
Queued
rabotiaga012-code wants to merge 1 commit into
Queued
rabotiaga012-code wants to merge 1 commit into
rabotiaga012-code wants to merge 1 commit into
Conversation
construction.ts (1 569 строк, setup на 814) разложен по папке modules/construction/: kit — общие правила, orders и events — регистрация приказов и событий. Код перенесён без правок, порядок регистрации прежний. Сторож трейта issued в schemas.test.ts теперь читает construction/orders.ts, где живут ворота заказа. Две записи сняты из code-size-budget.json. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013ZAJhv3ynVrGievU2aZ2Ua
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://claude-project-thread-h4dmn4-void-dominion.mazujlava.workers.dev (commit ea45eee)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
Contributor
🛡️ Security scan — сводный отчётКоммит: 🔎 Подтверждение сканов — 18/21 (+3 пропущено)
Подавлено сканерами (с обоснованием в коде/конфиге): 4 — в таблицы ниже не входят. По инструментам
Уже есть на Новые находки (топ 30 из 38)
…и ещё 8. Полные SARIF — в артефактах прогона. ℹ️ Блокирующие сканеры (SEC-1): Semgrep, Gitleaks, OSV, Trivy fs/image — находка или сбой скана валит их джобу; остальные (CodeQL, TruffleHog, Trivy deps, Dependency-Check, zizmor, Scorecard) — информационные. Разные движки/источники — для перекрёстной валидации; «0» достоверно только у подтверждённых сканеров. Полные SARIF/SBOM — в артефактах прогона. |
rabotiaga012-code
marked this pull request as ready for review
November 11, 2026 13:30
github-actions
Bot
added this pull request to the merge queue
Nov 11, 2026
Any commits made after this event will not be merged.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by My Lord · project thread
Что и зачем
Второй шаг после #2105 (челноки), сделан тем же способом. В
packages/shared-core/src/modules/construction.tsбыло 1 569 строк, аconstructionModule.setupзанимала 814 строк. Модуль разложен по уже существующей папкеmodules/construction/:construction.tssetupиз 5 вызововconstruction/kit.tsconstruction/orders.tsregisterBuildingOrders(возможности, ремонт, постройка, улучшение),registerUnitOrders(заказ юнитов, отмена, возобновление)construction/events.tsregisterCompletion(завершение, подкачка очереди, гибель станции, захват),registerWarDamage(прикрытие и износ построек)register*и отступ в них. Это проверено построчным сравнением со старым файлом.construction.tsреэкспортируетBuildingHit,BuildingDamage,DelayedOrder,ConstructionDelayи прежний реэкспорт изstate/buildFacilities.issuedвdata/schemas.test.tsчитал исходник ворот заказа сырым текстом. Теперь он читаетconstruction/orders.ts, где эти ворота живут.construction.tsиconstructionModule.setup.Зона
packages/shared-core/src/kernel/) не редактировалось под фичуКачество
pnpm run checkзелёный: 15 554 теста, на свежем mainДокументация
docs/state.mdне менялся: поведение прежнееGenerated by Claude Code