Now liveThe Skillselion MCP - thousands of ranked skills, loaded into your agent mid-task. No install.Get it →
hylarucoder avatar

Hai Ssot

  • 5 installs
  • 277 repo stars
  • Updated June 11, 2026
  • hylarucoder/hai-stack

Diagnoses single-source-of-truth violations across code seams and produces a numbered findings report with file:line evidence and a treatment-recipe table.

About

Hunts down single-source-of-truth violations across ten symptom classes (multi-source literals, shape proliferation, redundant conversions, re-implemented derivations, and more) and produces a numbered findings report with severity and treatment recipes. A developer uses it when they suspect duplicated definitions or drift between layers.

  • Detects ten SSOT symptom classes across type-system-unreachable seams
  • Routes each finding to a treatment recipe with honest false-positive adjudication

Hai Ssot by the numbers

  • 5 all-time installs (skills.sh)
  • +1 installs in the week ending Aug 2, 2026 (Skillselion tracking)
  • Ranked #881 of 1,352 Code Review & Quality skills by installs in the Skillselion catalog
  • Data as of Aug 2, 2026 (Skillselion catalog sync)
npx skills add https://github.com/hylarucoder/hai-stack --skill hai-ssot

Add your badge

Show developers this skill is listed on Skillselion. Paste this into your README.

Listed on Skillselion
Installs5
repo stars277
Last updatedJune 11, 2026
Repositoryhylarucoder/hai-stack

What it does

Diagnoses single-source-of-truth violations across code seams and produces a numbered findings report with file:line evidence and a treatment-recipe table.

Files

SKILL.mdMarkdownGitHub ↗

Hai SSOT

For Chinese readers, see SKILL.zh_CN.md. The English SKILL.md is the execution source of truth.

Overview

Hunt down places where one fact, one shape, or one word has more than one authoritative home — or where one name secretly serves several facts. Produce a findings report that an engineer can execute from: every finding numbered, evidenced, honestly adjudicated, and routed to a concrete disposition. The skill is a diagnostic with a strong opinion about treatment, not a linter.

The Core Law

SSOT violations cluster, almost without exception, at boundaries the type system cannot reach: language↔database (string literals in raw SQL and CHECK constraints), language↔wire (hand-written schemas mirroring the producing types), layer↔layer (untyped payload envelopes — Record<string, unknown>, map[string]any, bare dicts — keyed by raw strings), code↔docs, production↔fixture. Inside one compiler's reach, multi-source dies naturally — a second definition is a compile error or an obvious dead symbol. Outside it, multi-source is the equilibrium state: every producer hand-builds, every consumer hand-gropes, every layer re-copies the strings it needs.

Two corollaries that direct the hunt:

1. Start at the cross-stack seams; don't burn time grepping for intra-language duplication. 2. A test that pins two copies equal ("parity test", "pin test", "currency test") is a flag: either the copy should not exist (eliminate it, then delete the test), or both sides are genuinely real artifacts that cannot share a source (then the test IS the correct treatment). Locate every such test early — each one marks either a violation or a treatment already applied.

The Ten Symptom Classes

Read references/detection-cookbook.md for concrete search recipes per class before sweeping.

#SymptomOne-line definitionCanonical tell
1Multi-source literalsOne wire string / enum value defined independently in N placesSame quoted literal in two modules; a private const shadowing a public one
2Shape proliferationOne concept carried by N type/schema shapes across seamsHand-written schema on one side of a wire mirroring the producing type on the other; typed→untyped "regressions" where a typed object gets flattened back into a string-keyed map at a seam
3Word overloadOne word meaning N different things (the mirror of #1)The domain's most valuable word (e.g. "memory" as product moat) also used for infrastructure ("in-memory"); a term with 2-3 documented senses
4Legacy-vocabulary mapping layersOld vocabulary survives inside display/fixture mapping functions after a wire rename — and the mappings themselves get copy-pastedThe same old→new word map appearing in two files; fixtures asserting retired vocabulary
5Dual-pathway behavior forksThe same operation behaves differently depending on which entry path/host ran itA CLI path skipping the safety pipeline the server path runs; one caller passing explicit-empty where another gets defaults
6Scattered defaultsThe same fallback value born independently at multiple layersA default directory/timeout/limit defined as a constant in one layer AND as an inline fallback in another, neither referencing the other
7Pure-subset shape pairsType B = type A minus k fields, plus a field-by-field copy converterA converter function that only copies fields; the "information" carried by the second shape is merely hiding fields
8Same-name-different-shapeTwo exported types with the same name in the same semantic domain but different fieldsRunSnapshot in two sibling modules meaning related-but-different things
9Redundant conversion chainsOne concept reshaped at multiple hops along a single call path, where the intermediate shapes add no informationA value converted A→B→C on its way through layers; round-trips (A→B→A); typed→string→typed relays where a value is serialized and re-parsed inside one process. Kill question per hop: "what information does this shape add?" — no answer means the hop merges
10Re-implemented derivationsThe same rule — validation, normalization, parsing, a derived field — implemented independently at N layers, each a drift pointThe same regex/threshold/branching duplicated with small diffs; a date string parsed at three layers; isActive/displayName computed differently in two views

Severity logic: a violation that has already caused a production symptom (silent empty rendering, correctness bug, wrong cursor) outranks everything; next, violations on persisted or user-visible wire; then cross-team/cross-stack seams; intra-module duplication last.

Honest Adjudication — what NOT to flag

Findings are leads, not verdicts. Run these checks before a finding enters the report; record exonerated candidates in a "not counted" note so the next sweeper doesn't re-litigate:

  • Module-qualified generic names are idiomatic, not violations. A short generic type name

qualified by its module (stream.Message vs processor.Message) is the standard-library pattern in every module system (see the cookbook's language notes for per-language exemplars). Flag same-name types only when they share a semantic domain and confuse a cross-module reader (class #8), or collide in one file.

  • Forward contracts are alive even with zero producers. A registry entry / enum value with no

backend producer may be consumed by a frontend switch as a forward contract. Grep every consumer surface (web, contract, seeds) before calling anything dead. Disposition for these: annotate and group, never delete on producer-absence alone.

  • Persisted wire literals are frozen. The fix unifies definition sites; the string values

on disk/in events never change. Even naming inconsistencies baked into the wire (mixed prefixes) get documented, not repaired.

  • A shape change is legitimate when it carries information — adds a sequence number, hides

internal fields for an audience, renames into a consumer's vocabulary. The disease is reshaping that carries nothing (class #7), typed→map regressions, and chains where every hop re-converts without adding anything (class #9). Judge each hop separately: a chain can contain one real boundary and two gratuitous ones.

  • Re-checks at trust boundaries are defense, not duplication. A server re-validating client

input, or a DB constraint backing an app-level check, is deliberate redundancy across trust levels. Class #10 flags re-implemented rules at the same trust level — two layers behind the same boundary each owning their own copy of the regex, threshold, or parse.

  • Port/impl module pairs and per-plugin modules are conventions, not fragmentation. Don't

recommend flattening them in an SSOT report.

  • Deliberate, adjudicated dual vocabularies can exist (e.g. an ADR chose a flat result type

with a closed discriminator). Check decision records before flagging; contrast honestly — a 4-field type with a closed-set kind is not the same disease as a 9-field union with no discriminator semantics.

The credibility of the whole report rests on this section. One overreaching finding ("unify all the Messages!") teaches the reader to ignore the real ones.

Treatment Recipes

Every confirmed finding routes to exactly one recipe; the recipe determines the disposition:

RecipeWhenNotes
CodegenOne side can mechanically generate the other (type → schema, registry → enum file)Strongest fix; pairs with a currency test (guards "forgot to regenerate" — that is a constructive gap a compiler can't close, so the test is legitimate)
Parity guardBoth sides are real artifacts that cannot share a source (language enum vs DB CHECK constraint)Include a red drill: deliberately desync once and confirm the guard fires
Constant promotionBare string keys in map envelopes crossing layersPromote to a named constant next to its siblings; both writer and reader reference it
Typed payloadShape proliferation / map regressions at seamsOften a phase of a larger contract plan; don't band-aid per-field
Convert at the edgeRedundant conversion chainsConvert once where the value enters the system; pass one canonical type through the interior; merge hops that add no information
Single rule ownerRe-implemented derivationsHoist the rule into one named function/type and make every layer call it; where possible encode the proof in the type (parse, don't validate) so downstream layers cannot re-do the work
Vocabulary close-outWord overload, legacy mapping layersGlossary entry + rename of the cheap side; map function single-sourced or fixtures moved to current vocabulary
AdjudicationBehavior forksThese need a decision, not a patch: unify the behavior, or promote the fork into an explicitly documented contract. Present both options with a default recommendation
Delete the pinAfter any recipe eliminates a copy, delete the parity test that was holding the copies together — its survival is evidence of remaining multi-source

Workflow

1. Scope. Agree on the sweep surface (a module, a contract plane, the whole repo). Note any prior sweeps/plans to avoid re-finding adjudicated items. 2. Map the seams first. List the type-system-unreachable boundaries in scope (which wires, which DB constraints, which untyped envelopes, which generated artifacts). The Core Law says the findings live there. 3. Hunt per symptom class using the cookbook greps. For each candidate, capture file:line for every definition/use site — counts matter ("this literal is defined in exactly 2 places", "adding one event touches 6 files" is the change-amplification number that lands the point). 4. Verify producer AND consumer for anything you might call dead or removable. The three-surface discipline: backend producers, frontend/contract consumers, seeds/fixtures. 5. Adjudicate honestly (section above). Sort exonerated candidates into the "not counted" note. 6. Write the report using references/output-template.md: numbered findings, evidence, severity, recipe, disposition table, positive list ("already-healthy patterns to copy" — naming what the repo already does right makes the report constructive and gives fixes a local precedent to imitate). 7. Execute quick wins if asked — constant promotions and literal de-duplications are usually safe same-day (zero wire change, full test gate). Bigger recipes get routed to plans; behavior forks get routed to the user as decisions.

Hand off when

  • The finding's root cause is a module-boundary or layering problem → hai-architecture.
  • A finding needs a new name, or the report turns into a rename list → hai-naming.
  • The dispositions need to become a phased, verifiable plan → hai-goal.
  • The user wants to reframe the whole contract surface rather than patch findings → geju.

What this skill is NOT

  • Not a linter: it reports adjudicated findings with treatment routes, not raw matches.
  • Not "unify everything": its credibility comes from the not-counted list as much as the findings.
  • Not a wire-migration tool: persisted values are out of bounds; only definition sites move.

Related skills

This week in AI coding

Five minutes, every Monday - the tools, releases and tactics for developers.

unsubscribe anytime.