以下为本文档的中文说明该技能用于执行热核Thermo-Nuclear级别的代码审查是一种极其严格和深入的代码评审方法。它深入检查代码的每一处细节涵盖性能优化、安全性、可维护性和架构设计等各个方面。适用于对代码质量有极高要求的项目确保代码达到最高标准。热核审查的命名暗示了其审查强度和彻底性它不仅仅检查表面的代码风格问题而是深入到架构设计、性能瓶颈和安全缺陷等深层次问题。热核代码审查是一种极致的代码质量评审方法其严格程度远超常规的代码审查。该方法审查代码的每一个细节不仅仅关注功能和正确性还深入检查性能优化算法复杂度、内存分配模式、IO效率、安全性输入验证、认证授权、数据保护、可维护性代码组织、命名规范、注释质量和架构设计模块化、耦合度、扩展性。该方法适用于对可靠性、安全性和性能有极致要求的关键系统如金融交易系统、医疗设备和航天软件等领域。热核代码审查代表着代码质量评审的最高标准其名称源自核聚变反应的意象意味着审查过程要产生足够的高温和高压来检验代码的每一个原子级细节。该方法不仅仅关注代码的功能正确性更从多维度审视代码质量性能方面审查算法复杂度、内存分配和IO效率安全方面审查输入验证、认证授权和数据保护可维护性方面审查代码组织、命名规范和模块化程度。适用于对代码质量有极致要求的关键系统。Thermo-Nuclear Code Quality ReviewUse this skill for an unusually strict review focused on implementation quality, maintainability, abstraction quality, and codebase health.Above all, this skill should push the reviewer to beambitiousabout code structure. Do not merely identify local cleanup opportunities. Actively search for “code judo” moves: restructurings that preserve behavior while making the implementation dramatically simpler, smaller, more direct, and more elegant.Core PromptStart from this baseline:Perform a deep code quality audit of the current branch’s changes.Rethink how to structure / implement the changes to meaningfully improve code quality without impacting behavior.Work to improve abstractions, modularity, reduce Spaghetti code, improve succinctness and legibility.Be ambitious, if there is a clear path to improving the implementation that involves restructuring some of the codebase, go for it.Be extremely thorough and rigorous. Measure twice, cut once.Non-Negotiable Additional StandardsApply the baseline prompt above, plus these explicit review rules:Be ambitious about structural simplification.Do not stop at “this could be a bit cleaner.”Look for opportunities to reframe the change so that whole branches, helpers, modes, conditionals, or layers disappear entirely.Prefer the solution that makes the code feel inevitable in hindsight.Assume there is often a “code judo” move available: a re-organization that uses the existing architecture more effectively and makes the change dramatically simpler and more elegant.If you see a path to delete complexity rather than rearrange it, push hard for that path.Do not let a PR push a file from under 1k lines to over 1k lines without a very strong reason.Treat this as a strong code-quality smell by default.Prefer extracting helpers, subcomponents, modules, or local abstractions instead of letting a file sprawl past 1000 lines.If the diff crosses that threshold, explicitly ask whether the code should be decomposed first.Only waive this if there is a compelling structural reason and the resulting file is still clearly organized.Do not allow random spaghetti growth in existing code.Be highly suspicious of new ad-hoc conditionals, scattered special cases, or one-off branches inserted into unrelated flows.If a change adds “weird if statements in random places”, treat that as a design problem, not a stylistic nit.Prefer pushing the logic into a dedicated abstraction, helper, state machine, policy object, or separate module instead of tangling an existing path.Call out changes that make the surrounding code harder to reason about, even if they technically work.Bias toward cleaning the design, not just accepting working code.If behavior can stay the same while the structure becomes meaningfully cleaner, push for the cleaner version.Do not rubber-stamp “it works” implementations that leave the codebase messier.Strongly prefer simplifications that remove moving pieces altogether over refactors that merely spread the same complexity around.Prefer direct, boring, maintainable code over hacky or magical code.Treat brittle, ad-hoc, or “magic” behavior as a code-quality problem.Be skeptical of generic mechanisms that hide simple>Primary Review QuestionsFor every meaningful change, ask:Is there a “code judo” move that would make this dramatically simpler?Can this change be reframed so fewer concepts, branches, or helper layers are needed?Does this improve or worsen the local architecture?Did the diff add branching complexity where a better abstraction should exist?Did a previously cohesive module become more coupled, more stateful, or harder to scan?Is this logic living in the right file and layer?Did this change enlarge a file or component past a healthy size boundary?Are there repeated conditionals that signal a missing model or missing helper?Is the implementation direct and legible, or does it rely on special cases and incidental control flow?Is this abstraction actually earning its keep, or is it just a wrapper?Did the diff introduce casts, optionality, or ad-hoc object shapes that obscure the real invariant?Is this logic living in the canonical layer, or did the diff leak details across a boundary?Is this orchestration more sequential or less atomic than it needs to be?What to Flag AggressivelyEscalate findings when you see:A complicated implementation where a cleaner reframing could delete whole categories of complexity.Refactors that move code around but fail to reduce the number of concepts a reader must hold in their head.A file crossing 1000 lines due to the PR, especially if the new code could be split out.New conditionals bolted onto unrelated code paths.One-off booleans, nullable modes, or flags that complicate existing control flow.Feature-specific logic leaking into general-purpose modules.Generic “magic” handling that hides simple structure and makes the code harder to reason about.Thin wrappers or identity abstractions that add indirection without simplifying anything.Unnecessary casts,any,unknown, or optional params that muddy the real contract.Copy-pasted logic instead of extracted helpers.Narrow edge-case handling implemented in the middle of an already busy function.Refactors that technically pass tests but make the code less modular or less readable.“Temporary” branching that is likely to become permanent debt.Bespoke helpers where the codebase already has a canonical utility for the job.Logic added in the wrong layer/package when it should live somewhere more central.Sequential async flow where obviously independent work could stay simpler and clearer with parallel execution.Partial-update logic that leaves state less atomic than necessary.Preferred RemediesWhen you identify a code-quality problem, prefer suggestions like:Delete a whole layer of indirection rather than polishing it.Reframe the state model so conditionals disappear instead of getting centralized.Change the ownership boundary so the feature becomes a natural extension of an existing abstraction.Turn special-case logic into a simpler default flow with fewer exceptions.Extract a helper or pure function.Split a large file into smaller focused modules.Move feature-specific logic behind a dedicated abstraction.Replace condition chains with a typed model or explicit dispatcher.Separate orchestration from business logic.Collapse duplicate branches into a single clearer flow.Delete wrappers that do not meaningfully clarify the API.Reuse the existing canonical helper instead of introducing a near-duplicate.Make type boundaries more explicit so the control flow gets simpler.Move the logic to the package/module/layer that already owns the concept.Parallelize independent work when that also simplifies the orchestration.Restructure related updates into a more atomic flow when partial state would be harder to reason about.Do not be satisfied with “maybe rename this” feedback when the real issue is structural.Do not be satisfied with a merely cleaner version of the same messy idea if there is a plausible path to a much simpler idea.Review ToneBe direct, serious, and demanding about quality.Do not be rude, but do not soften major maintainability issues into mild suggestions.If the code is making the codebase messier, say so clearly.If the implementation missed an opportunity for a dramatic simplification, say that clearly too.Good phrases:this pushes the file past 1k lines. can we decompose this first?this adds another special-case branch into an already busy flow. can we move this behind its own abstraction?this works, but it makes the surrounding code more spaghetti. lets keep the behavior and restructure the implementation.this feels like feature logic leaking into a shared path. can we isolate it?this abstraction seems unnecessary. can we just keep the direct flow?why does this need a cast / optional here? can we make the boundary more explicit instead?this looks like a bespoke helper for something we already have elsewhere. can we reuse the canonical one?i think theres a code-judo move here that makes this much simpler. can we reframe this so these branches disappear?this refactor moves complexity around, but doesnt really delete it. is there a way to make the model itself simpler?Output ExpectationsPrioritize findings in this order:Structural code-quality regressionsMissed opportunities for dramatic simplification / code-judo restructuringSpaghetti / branching complexity increasesBoundary / abstraction / type-contract problems that make the code harder to reason aboutFile-size and decomposition concernsModularity and abstraction issuesLegibility and maintainability concernsDo not flood the review with low-value nits if there are larger structural issues.Prefer a smaller number of high-conviction comments over a long list of cosmetic notes.Approval BarDo not approve merely because behavior seems correct.The bar for approval is:no clear structural regressionno obvious missed opportunity to make the implementation dramatically simpler when such a path is visibleno unjustified file-size explosionno obvious spaghetti-growth from special-case branchingno obviously hacky or magical abstraction that makes the code harder to reason aboutno unnecessary wrapper/cast/optionality churn obscuring the real designno clear architecture-boundary leak or avoidable canonical-helper duplicationno missed opportunity for an obvious decomposition that would materially improve maintainabilityTreat these as presumptive blockers unless the author can justify them clearly:the PR preserves a lot of incidental complexity when there is a plausible code-judo move that would delete itthe PR pushes a file from below 1000 lines to above 1000 linesthe PR adds ad-hoc branching that makes an existing flow more tangledthe PR solves a local problem by scattering feature checks across shared codethe PR adds an unnecessary abstraction, wrapper, or cast-heavy contract that makes the design more indirectthe PR duplicates an existing helper or puts logic in the wrong layer when there is a clear canonical homeIf those conditions are not met, leave explicit, actionable feedback and push for a cleaner decomposition.