agentsclimarketplace

Threat model surface

Skill alpha-omega-security/threat-model/skills/threat-model-surface

Agent skill for producing threat models for open-source projects

Install
npx -y skills add alpha-omega-security/threat-model --skill threat-model-surface

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

2 things to look at

  • 28 days oldThe repository was created 28 days ago. New is not bad, but a brand new repository carrying a familiar-sounding name is the shape a typosquat arrives in, and there has been no time for anyone else to find a problem with it.
  • 5 stars5 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Perform phase 3.3 deep analysis of an in-scope attack surface for a threat model. USE WHEN the orientation brief is ready and code must be read to derive the §1.7 per-input trust table and contract-dimension matrix, §1.5 no-surprise side-effects inventory, §1.4 reachability preconditions, and §1.8 output taint. Reads public entry points for contract rather than defects, timeboxes each component family, and marks uncovered surface inferred. Produces a surface analysis. Read-only. DO NOT USE FOR: bug hunting, code review, orientation, or drafting the prose model.

SKILL.md

9.6 KB, as published. Nobody here has run it

Threat Model — Surface (deep pass on the in-scope surface)

Phase 3.3. The orient pass was minutes; this is hours, and that is expected. Three of the output's most valuable artifacts cannot be produced any other way:

  • the per-input-operand trust table (§1.7), which requires reading each in-scope entry point far enough to say which direct parameters and indirect inputs an attacker can reach and what the caller must enforce; and
  • the contract-dimension matrix (§1.7-§1.12), which prevents silence about failure behavior, representational limits, executable collaborators, object topology, and lifecycle edge cases from becoming downstream MODEL-GAP findings; and
  • the no-surprise side-effects inventory (§1.5) — negative claims about what the project does to its host that cannot be established by reading docs.

Read principles.md and the §1.5 / §1.7 / §1.8 specs in output-structure.md first.

Rules for keeping the cost bounded

  • Scope by the recon carve. Read only the entry points of in-model families. Do not read contrib/, examples, or out-of-scope families beyond confirming they are separable.
  • Read for contract, not for bugs. At each entry point the question is "which of these parameters can an attacker control, what kind of control is it, and what contract applies at edge conditions?" — not "is this code correct?" Record whether behavior is guaranteed, disclaimed, or unresolved; do not test whether the implementation satisfies it. The moment the reading turns into review, stop and move on.
  • Timebox per family. If a family's surface is too large to table in budget (e.g., a service with 100+ routes), table the highest-exposure subset, mark the remainder (inferred, QN) with a coverage note, and raise completing the table as an open question / follow-up — do not silently generalize.
  • Record hypotheses as you go, in draft form with provenance tags. Preserve (documented) provenance for explicit normative public contracts. Code, implementation comments, and tests that merely suggest an unwritten contract remain (inferred, QN) until a maintainer ratifies them.
  • Cite harder before tagging inferred. Before marking a row (inferred), check the API docs, header comments, Javadoc/package-info, manpage, and README — a fact stated there is (documented, source), not inferred. Turning a false-inferred into a true-documented row is pure accuracy and directly reduces the escalation count.
  • Disclaim demonstrably-absent guarantees rather than leaving them open. When the reading shows a family makes no thread-safety, resource-bound, or failure-atomicity guarantee, that absence is verifiable — record the matrix row as disclaimed with (documented, source), routing to §1.12, not as unresolved. Reserve unresolved / (inferred) for dimensions where a guarantee plausibly exists but you could not confirm it. Where you must reason past the verifiable to a clear safe default, tag (assumption, QN) rather than (inferred, QN).

Build the per-input-operand trust table (§1.7)

One row per direct parameter and each security-relevant indirect input of every public entry point:

FunctionInput operandAttacker-controllable?Control kindCaller must enforceProvenance
gzopenpathno — trusted caller stringdata, resource-namepath sanitization(documented, gzopen contract)
gzreadfile contentsyesdata, sizeoutput buffer >= len(inferred, Q4)
gzprintfformatno — trusted literalx-format-stringnever source from input(inferred, Q5)

For a network service, the first column is the route/endpoint or protocol message (POST /v1/configuration, Handshake frame), and rows must cover headers and connection metadata as well as bodies — header-presence checks (X-Forwarded-*, auth tokens) are common false friends. Group rows by component family if the table grows large. Prose is not sufficient: tool/AI findings are reported against specific sinks, and the triager must look up the exact parameter.

Do not collapse all control into a boolean. Use one or more control kinds: data, size/rate, type/class, callback/code, object-graph topology, collaborator implementation, resource-name, serialized state, or a project-specific x- kind. Distinguish an attacker choosing data passed through a trusted callback from an attacker choosing the callback itself.

Also capture size/shape/rate assumptions (bounded? streaming? memory-mapped?), and flag any input whose magnitude drives resource allocation (memory, threads, file handles) — that feeds the §1.11 resource-property threshold.

Build the contract-dimension matrix (§1.7-§1.12)

For every in-scope component family, fill every applicable row. A blank cell is not allowed:

DimensionStatusConditions / boundaryRoutes toProvenance
numeric domain and representational limitsclaimed / disclaimed / N/A / unresolvedmaximum size, overflow behavior, normalization domain§1.11 / §1.12 / §1.18(inferred, QN) or cited documented/maintainer source
failure and exception atomicityclaimed / disclaimed / N/A / unresolvedstate after validation or callback failure§1.11 / §1.12 / §1.18(inferred, QN) or cited source
recursive or cyclic topologyclaimed / disclaimed / N/A / unresolvedself-reference, graph depth, reentrancy§1.11 / §1.12 / §1.18(inferred, QN) or cited source
callback and collaborator executionclaimed / disclaimed / N/A / unresolvedcomparator, predicate, factory, virtual dispatch§1.7 / §1.10 / §1.11 / §1.12 / §1.18(inferred, QN) or cited source
serialization and reconstructionclaimed / disclaimed / N/A / unresolvedrestored types, callbacks, invariant rebuilding§1.3 / §1.7 / §1.10 / §1.11 / §1.12 / §1.18(inferred, QN) or cited source
reference and object lifecycleclaimed / disclaimed / N/A / unresolvedweak/soft references, GC clearing, invalidation§1.5 / §1.11 / §1.12 / §1.18(inferred, QN) or cited source
concurrency and reentrancyclaimed / disclaimed / N/A / unresolvedshared mutation, callback reentry§1.5 / §1.11 / §1.12 / §1.18(inferred, QN) or cited source
resource complexityclaimed / disclaimed / N/A / unresolvedCPU, heap, stack, I/O as a function of input/state§1.7 / §1.11 / §1.12 / §1.18(inferred, QN) or cited source

Add project-type rows when needed, such as Unicode/canonicalization, probabilistic-result semantics, protocol state transitions, clock behavior, or distributed consistency. The matrix is a contract inventory, not a bug list. A demonstrably-absent guarantee is a disclaimed row with (documented) provenance; only a dimension where a guarantee might exist stays unresolved and becomes a proposed-answer question for the interview.

For stateful APIs, explicitly record the postcondition on failure: unchanged, partially committed, best-effort cleanup, or unspecified. Cover failures from validation, allocation, caller callbacks, and delegated collaborator methods.

Build the no-surprise side-effects inventory (§1.5)

Scan for what the project does to its host, then state the negative claims: does it open sockets? spawn processes? install signal handlers? read environment variables? write to stdout/stderr? touch global locale or FPU state? mutate process-wide state? These are rarely documented and almost impossible to cite, so they will be predominantly (inferred, QN) — flag them as wave-1/2 confirmation targets for the interview.

Derive reachability preconditions (§1.4) and output taint (§1.8)

  • Per in-model family, state the condition a finding must meet to matter ("a finding in inflate.c is in-model only if reachable from the compressed input bytes"). This is the first test a triager applies to a tool/AI hit.
  • Per output channel, state taint. The default for parsers/decoders/decompressors is one line worth stating verbatim: "Output is exactly as untrusted as the input it derives from; no sanitization, normalization, or encoding is performed." Note any structural invariant the code actually upholds (bounded writes, valid encoding, matching length fields) — each is a candidate §1.11 property.

Output — surface analysis

Hand back: the §1.7 table (with coverage note if partial), the completed contract-dimension matrix, the §1.5 side-effects inventory, per-family §1.4 reachability preconditions, and §1.8 output-taint statements — each tagged with its actual provenance. Mark code-derived contract hypotheses (inferred, QN) and call out unresolved dimensions and wave-1/2 confirmation targets for threat-model-interview.

Keep looking

Skills are one crate of 328,083. Ordering is by how many stacks a row turns up in, so the top of any crate is what has actually been picked rather than what has the most stars.