@azure-id/orc 1.4.2 → 1.6.0

package/mock-run/orc-test.md

@@ -0,0 +1,327 @@
+# Mock run — `/orc-test`
+
+> ORC runs the test against a real system, writes down what it saw, and never touches the system it is testing.
+
+---
+
+## 1. What it does
+
+Every other testing tool in ORC **writes** tests. `/orc-test` **runs** them.
+
+It points at a running API — yours, on your laptop, or one you were given
+permission to touch — sends real requests, and writes down what came back. Then
+it stops.
+
+The split is the whole design:
+
+> **The CLI executes and measures. The model designs and interprets.**
+> A model never sends a request. The CLI never decides what a response means.
+
+That split is what makes the report worth reading. A number in it was measured
+by a program. A sentence in it was written by a model that only ever saw
+evidence on disk.
+
+Three rules hold it together, and each one is a contract with a name:
+
+| Rule | What it means |
+|---|---|
+| `a lane that reports a result it did not observe` | Three verdicts: `pass`, `fail`, `unknown`. **`unknown` is the honest one.** A case nobody ran is never a pass. |
+| `a lane that sends traffic to a target nobody authorized` | The target is frozen at the start, with a written statement of who said yes. Nothing outside that origin is ever contacted. |
+| `a lane that fixes the system under test` | ORC never edits the thing it is testing. When the app is broken, it hands the machine back to you. |
+
+---
+
+## 2. Freeze the target
+
+Nothing here has a default it could guess. There is no config key that names a
+target, on purpose.
+
+```
+$ orc test init api-users --kind be --env local \
+    --base-url http://127.0.0.1:4000 \
+    --authorized "me, on my own laptop, against my own dev database" \
+    --destructive deny --reason "this box shares a database with the seed script"
+```
+
+```
+  /orc-test · api-users
+  target FROZEN at orc/orc-test/api-users/target.md
+
+    kind          be
+    environment   local
+    base URL      http://127.0.0.1:4000
+    origin fence  http://127.0.0.1:4000
+    destructive   deny — this box shares a database with the seed script
+
+  This folder is NEVER staged. It will hold real response bodies from a real
+  system. Add this line to .gitignore yourself — ORC does not edit it:
+      orc/orc-test/
+
+  Next:  orc test surface api-users
+```
+
+Two things to notice.
+
+**The origin fence.** Every request the runner sends is checked against it. A
+redirect that leaves the fence is not followed — it is recorded and dropped.
+
+**`--authorized` is a sentence, not a checkbox.** ORC records it word for word
+and never checks it. On a remote target it is required. It prints at the top of
+every report, so the person reading the report can see whose permission this
+was run on.
+
+---
+
+## 3. The free pass
+
+The surface costs nothing. No model, no tokens.
+
+```
+$ orc test surface api-users
+```
+
+```
+  /orc-test · api-users · surface
+  the free pass — zero model tokens. Written to orc/orc-test/api-users/surface.md
+
+    spec on disk   openapi.yaml  (4 routes)
+    spec at target http://127.0.0.1:4000/openapi.json
+    frameworks     express
+    routes         5   (34 files scanned, 1 live-only)
+    mounted at     /api/v1
+    unresolved     1
+
+    in code, not live   1   dead or unshipped
+    live, not in code   1   shadow / zombie APIs — OWASP API9:2023
+        GET /internal/debug
+```
+
+`GET /internal/debug` answers at the target and is in no file in this
+repository. Nobody reviews it. Nobody patches it. Finding that took zero model
+tokens and one comparison of two documents — **no route was called to produce
+it**.
+
+If there had been no spec at the target, that line would read:
+
+```
+    code-vs-live diff   NOT MEASURED — no spec answered at the target, so there
+    is nothing to compare the code against. This is NOT 'no shadow APIs' — it is
+    'not measured'.
+```
+
+That distinction is the lane in one sentence.
+
+---
+
+## 4. When the app is broken
+
+```
+$ orc test env api-users
+```
+
+```
+  /orc-test · api-users · environment
+  state  UNHEALTHY   the process is up and answering wrong. ORC does not fix
+                     the system it is testing.
+
+    detected   npm run dev   (package.json)
+    process    pid 48211
+    missing    STRIPE_SECRET_KEY, SESSION_SECRET
+
+    Error: password authentication failed for user "app"
+```
+
+And then it stops.
+
+It found the start command. It knows the process is alive. It can see the exact
+line in the log. It will not open that file. It names the two environment keys
+that are absent and **writes no value for either** — a placeholder written into
+a real environment is a credential-shaped lie, and only you know the right
+value.
+
+You fix it. You run the command again. That is the whole loop.
+
+---
+
+## 5. The cases derive themselves
+
+```
+$ orc test case derive api-users
+```
+
+```
+  /orc-test · api-users · cases
+  derived for free — zero model tokens
+
+    happy     2
+    edge      4
+    abuse     3
+    security  3
+    total     12
+
+  2 cases carry a GAP — what a schema could not answer.
+  This is what the designer agent is for, and it is the only part that costs anything.
+
+  security tier  safe   (the shipped default)
+  5 OWASP rows UNCHECKABLE — each keeps its slot, and none of them will ever become a pass
+```
+
+The matrix comes out of the schema for free. The **gaps** are what a schema
+cannot know — what a valid email looks like in this business, which order the
+two calls have to happen in. Those go to the designer agent, and that is the
+only part of this lane you pay for.
+
+---
+
+## 6. The security tier is a closed set
+
+```
+$ orc test security api-users
+```
+
+```
+  /orc-test · api-users · security · OWASP API Top 10 (2023)
+  tier safe (the shipped default) · the set is CLOSED and is never extended ad hoc
+
+    API1   FOUND           Broken object level authorization (BOLA)
+           severity critical  OBSERVED, and the request crossed an identity boundary
+           C-006  replay a successful request as a SECOND identity
+           orc/orc-test/api-users/runs/03/evidence/C-006/
+    API2   observed-clean  Broken authentication
+    API5   unchecked       Broken function level authorization (BFLA)
+           only one ROLE is declared. BFLA asks whether a caller who should be
+           REFUSED is refused; with one role there is nobody to refuse.
+    …
+
+  DETECTION, NEVER EXPLOITATION. Every probe demonstrates the CONDITION and stops there.
+  `unchecked` keeps its slot, never becomes a pass, and never raises the exit code.
+```
+
+**Ten rows, always, in every state.** A category ORC could not measure says so
+and says why. It never becomes a pass, and it never changes the exit code.
+
+That last rule matters more than it looks. Reporting an unmeasured category as
+clean is the single most damaging thing a security report can do, because
+somebody then ships on it.
+
+---
+
+## 7. The run
+
+```
+$ orc test run api-users
+```
+
+The ladder runs `happy` → `edge` → `abuse` → `security`, and **it stops on a red
+happy path**. A 500 on every request will "prove" a dozen vulnerabilities that
+are one bug, so there is no point continuing.
+
+```
+  /orc-test · api-users · run 3
+  2 req/s, concurrency 2 · X-Orc-Test-Run: api-users-03 · written to runs/03
+
+    pass      4
+    fail      3
+    unknown   4
+
+  The ladder STOPPED at the edge tier. The cases below it were not observed,
+  and none of them is a pass.
+```
+
+Four things the runner does that you should know about:
+
+- **The pace is the target's, not ORC's.** A fixed small concurrency and a rate
+  limit the CLI actually applies.
+- **A 429 is a result.** It means rate limiting works. ORC records it, backs
+  off, and does not push through — pushing through is how a staging scan becomes
+  a lockout.
+- **Failed logins are capped** per identity, so a run cannot lock somebody's
+  account out.
+- **Redaction is structural.** Every credential is replaced *before* the bytes
+  reach disk, not in a review step afterwards.
+
+---
+
+## 8. The report
+
+```
+$ orc test report api-users
+```
+
+```
+    cases     12   4 pass · 3 fail · 4 not observed · 1 never run
+    runs      3
+    findings  3    2 observed · 1 not observed, and none of those is a pass
+    security  safe   2 FOUND
+    flakes    1      recorded, never retried away — the instability IS the finding
+
+  This folder is NEVER staged.
+```
+
+A finding must cite evidence that resolves on disk, or it is **dropped by name
+and counted**. A model that describes a vulnerability it cannot point at does
+not get to put it in your report.
+
+Severity is ORC's. If the interpreting model disagrees, its word is kept beside
+ORC's and printed — so a disagreement is visible instead of invisible.
+
+And a flake is written down, never retried away. If the same case answers
+differently on two runs, **that instability is the finding.**
+
+---
+
+## 9. What lands on disk
+
+```
+orc/orc-test/api-users/
+  test.json        the ledger — the single source of truth
+  target.md        the frozen target, and who authorized it
+  surface.md       the routes, and the code-vs-live diff
+  cases.json       the matrix (derived from the ledger)
+  findings.md      the findings (derived from the ledger)
+  REPORT.md        the one file written to be shared
+  runs/03/evidence/C-006/
+     request.txt   redacted
+     response.txt  redacted
+     repro.sh      a curl you can run yourself, with the secret taken out
+```
+
+**Never stage this folder.** It holds real response bodies from a real system —
+the most sensitive thing ORC writes to disk. `orc doctor` tells you if it is not
+git-ignored, and it will not edit `.gitignore` for you.
+
+---
+
+## 10. In the panel
+
+`orc ui` ▸ **Test** shows the same thing, and works out none of it:
+
+- **Targets** — the frozen target, who authorized it, the identities, the
+  environment.
+- **Surface** — the routes, and the code-vs-live diff.
+- **Cases** — the matrix, and all ten OWASP rows.
+- **Runs** — one row per run, opening in place.
+- **Findings** — observed and not observed, kept apart.
+
+A free action is a button there. `orc test run` is not: it sends real traffic,
+so the panel gives you the command to copy and never a button.
+
+---
+
+## 11. What it will not do
+
+- It will not edit the system it is testing. Not a config file, not a
+  `.env`, not a line of source.
+- It will not send a request to anything outside the frozen origin.
+- It will not report a category it could not measure as clean.
+- It will not confirm an exploit. It shows the condition and stops there —
+  going further is a human's decision on a human's authority.
+- It will not stage or commit anything.
+
+---
+
+## Related
+
+- `orc-quick` — a fast fix once you know what is wrong.
+- `orc-challenge` — grade the report itself, in a different session.
+- `orc-pact` — record the invariant a finding taught you.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azure-id/orc",
-  "version": "1.4.2",
+  "version": "1.6.0",
   "description": "ORC — an orchestrator skill constellation for Claude Code: intake, planning, scored parallel subagents, code-pattern matching, review, verify, ship, plus a project knowledge-base wiki.",
   "bin": {
     "orc": "bin/cli.js"

package/templates/agents/MODEL-MAPPING.md

@@ -58,6 +58,8 @@ model change, and an agent's model change is always a RENAME.
 | orc-challenge-expansionist-opus-5-med | claude-opus-5 | medium | THE COUNCIL: the only lens not looking for a defect — what is being undervalued? Returns `opportunity` objects with a first step and a route, never a severity (/orc-challenge only; read-only). MEDIUM because it is pattern work against a concrete artifact, the same class as the advisor |
 | orc-doc-writer-opus-5-med | claude-opus-5 | medium | write ONE part file of a long document from a slice of sections (/orc-doc only). It writes to `.work/<id>.md` and NEVER opens document.md — that is what makes parallel writing safe here |
 | orc-doc-checker-opus-5-low | claude-opus-5 | low | read ONE LINE RANGE of a document and report anchored findings (/orc-doc only; read-only). LOW ON PURPOSE — the same reasoning that pins the challenge cold reader: a harder-thinking checker reasons its way past a gap a real reader would trip on |
+| orc-test-designer-opus-5-high | claude-opus-5 | high | fill the GAPS the free case matrix could not derive, and author the front-end journey SCRIPT (/orc-test only). HIGH IS THE INSTRUMENT: a shallow designer returns the three obvious cases the deterministic expansion already covered, which is the one output that costs money and adds nothing |
+| orc-test-interpreter-opus-5-low | claude-opus-5 | low | read ONLY a run’s evidence files and say whether what is in them is a finding (/orc-test only; SEALED slice, `Read` and nothing else). LOW IS A MEASUREMENT CHOICE, NOT A COST ONE, and nothing may ever upgrade it: a harder-thinking interpreter reasons its way to why a leaked stack trace is probably fine in staging |
 | orc-trace-writer-haiku-4-5 | claude-haiku-4-5 | — (no ladder) | append one phase block of behavior-trace narration from an orchestrator packet (every trace-owning lane; append-only, never reads source) |
 
 ## Opus-5-only mode agents (hard-gated; dispatched only when `opus5_only: true`)

package/templates/agents/orc-test-designer-opus-5-high.md

@@ -0,0 +1,158 @@
+---
+name: orc-test-designer-opus-5-high
+description: >
+  ORC Test designer — claude-opus-5, high effort. Single-role: fill the GAPS the
+  free case matrix could not derive, and author the front-end journey SCRIPT. It
+  is dispatched only for what a schema cannot know — a semantically valid body,
+  which field is the object id a BOLA probe should swap, what "logged in" looks
+  like here — because the CLI already derived every partition, boundary and
+  negative for zero tokens. HIGH EFFORT IS THE INSTRUMENT: a shallow designer
+  returns the three obvious cases the deterministic expansion already covered,
+  which is the one output that costs money and adds nothing. It never sends a
+  request, never runs a browser, and never decides what a response means.
+  Dispatched by the orc-test skill at T6 and at the front-end half.
+model: claude-opus-5
+effort: high
+tools: Read, Write, Glob, Grep
+---
+
+You are the ORC Test designer (Opus 5, high effort).
+
+**The CLI EXECUTES and MEASURES. You DESIGN.** You never send a request, you
+never launch a browser, and you never decide what a response means. Everything
+you produce is a FILE the CLI reads back through a validating command.
+
+## Why you are expensive, and what that buys
+
+The free matrix already exists before you are dispatched. `orc test case derive`
+produced, at zero model tokens:
+
+- one happy case per target, from the declared schema
+- equivalence partitions per parameter — invalid type, null, empty, absent
+- boundary values — one below the minimum, the minimum, the maximum, one above,
+  `minLength`/`maxLength`, a value outside the enum
+- method and content-type negatives, and a malformed body
+- stateful create → read back → change → remove sequences
+- the whole closed OWASP set the run's tier allows
+
+**So a case you return that any of those already covers is waste with a price
+tag on it.** You are dispatched for `gaps[]` and for nothing else. Read them
+first; they are the job.
+
+High effort is the instrument here for exactly that reason: the cheap answer to
+"design some test cases" is the list the CLI already generated.
+
+## What you are given
+
+```
+role:              cases | journey
+target.md          the FROZEN target — kind, env, base URL, authorization, destructive decision
+surface.md         the routes, with in_code / live per row
+flow/<target>.md   entry → middleware → handler → calls → writes → preconditions → object_ids
+cases.json         the derived matrix, WITH `gaps[]` on the rows that carry one
+selector_policy    testid-ok | role-first          (journey role only)
+identities         name + role only — NEVER a credential
+```
+
+You may `Read`, `Glob` and `Grep` the repository. You may not run anything.
+
+## role: cases
+
+Return a JSON array to a file. The CLI reads it with
+`orc test case add <slug> --from <file.json>`, and **a row that fails validation
+is REFUSED BY NAME and nothing half-formed is written** — so a malformed row
+costs the whole dispatch, not just itself.
+
+```json
+[
+  {
+    "target": "POST /orders",
+    "tier": "happy",
+    "why": "a body that is semantically valid, not merely schema-valid: line items must reference a product that exists, and the total must match their sum",
+    "request": { "method": "POST", "path": "/orders", "headers": {"content-type":"application/json"},
+                 "body": { "...": "..." } },
+    "expect": { "status": [201] },
+    "identity": "user_a",
+    "mutates": true
+  }
+]
+```
+
+Rules, each of which the CLI enforces or the report depends on:
+
+1. **`target` must be a `key` that is in `surface.routes[]`.** A target ORC
+   accepted and could not find is a run that quietly tests nothing.
+2. **`tier` is one of `happy | edge | abuse | security`.** There is no fifth.
+3. **`why` is required and is written for a person reading the report in three
+   weeks.** A case nobody can explain later is a case nobody should run.
+4. **`expect.status` is a list, and it is the narrowest honest list.** `[200]`
+   when only 200 is right; `[200, 204]` when both genuinely are. A wide list
+   passes for the wrong reason.
+5. **Never invent a credential, a token, a real customer id or a real email.**
+   If a case needs a real object id, say so in `why` and leave the field as the
+   schema's example — the CLI already names that as a gap, and a plausible
+   invented id makes a case pass for the wrong reason.
+6. **`mutates: true` on anything that writes.** The runner will not send a
+   mutating case unless the run was initialised `--destructive allow`, and
+   getting this wrong in the cheap direction sends a write nobody authorised.
+7. **Do not re-derive the security tier.** The OWASP set is CLOSED and the CLI
+   owns it. If you believe a probe is missing, say so in your summary; do not
+   add a row with an `owasp` tag.
+
+Answer the gaps in order and stop. Ten cases that close ten named gaps beat
+forty that close three.
+
+## role: journey
+
+Author a Playwright spec, as a FILE. The CLI runs it with
+`orc test ui journey <slug> add --name <n> --from <file.js>` and then
+`orc test ui run`.
+
+**A step an LLM took is not a step you can re-run.** That is why you write a
+script instead of driving a browser: the script stays on disk, and re-running it
+is free and identical.
+
+```js
+const { test, expect } = require("@playwright/test");
+
+test("a customer can complete checkout", async ({ page }) => {
+  await page.goto("/cart");
+  await expect(page.getByRole("button", { name: "Checkout" })).toBeVisible();
+  // …
+});
+```
+
+Rules:
+
+1. **It must contain a `test(` call.** A spec with none runs, reports nothing,
+   and is indistinguishable from a journey that passed. The CLI refuses it.
+2. **Honour the `selector_policy` you were handed.**
+   - `testid-ok` (LOCAL) — a `data-testid` derived from source is trustworthy,
+     because the source is what is running.
+   - `role-first` (REMOTE) — role and accessible name FIRST, always. A
+     source-derived testid may be wrong because the deployed build is not this
+     commit, and the CLI reports that as a BUILD MISMATCH before it is treated
+     as a defect.
+3. **Never write a credential into the script.** Login is performed by
+   `orc test ui login`, which saves a storage state; your journey starts already
+   logged in. Read a value from `process.env` if you truly need one.
+4. **No `test.setTimeout` marathons and no retry loops.** There are no retries
+   in the generated config, on purpose: a flake is recorded, never retried away,
+   and the instability IS the finding.
+5. **Assert something a person would care about**, not that the page loaded.
+   `expect(page.getByText("Order #")).toBeVisible()` is a result; a screenshot
+   is not.
+6. **When `process.env.ORC_TEST_REPLAY_HAR` is set**, call
+   `await page.routeFromHAR(process.env.ORC_TEST_REPLAY_HAR, { update: false, notFound: "abort" })`
+   before the first `goto`. That run is for isolating a flake; the CLI records
+   NO result from it either way.
+
+## What you return
+
+A short summary, and the path of the file you wrote. Never the file's contents
+inline — the CLI reads the file, and a body that travelled through a summary is
+a body somebody retyped.
+
+State plainly which gaps you could NOT close and why. A gap you left open and
+named is a fact the report can carry; a gap you filled with a guess is a case
+that passes for the wrong reason, and nobody downstream can tell the difference.

package/templates/agents/orc-test-interpreter-opus-5-low.md

@@ -0,0 +1,130 @@
+---
+name: orc-test-interpreter-opus-5-low
+description: >
+  ORC Test interpreter — claude-opus-5, low effort. Single-role: read ONLY the
+  evidence files a run wrote — request.txt, response.txt — and say whether what
+  is in them is a finding. Its slice is SEALED: no project source, no session
+  prose, no diff summary, and it never re-sends anything. LOW EFFORT IS A
+  MEASUREMENT CHOICE, NOT A COST ONE, and nothing may ever upgrade it: a
+  harder-thinking interpreter reasons its way to why a leaked stack trace is
+  probably fine in staging, which is exactly the gap this instrument exists to
+  find. Every finding it returns cites an evidence path, or `orc test record`
+  DROPS it by name. Dispatched by the orc-test skill at T8.
+model: claude-opus-5
+effort: low
+tools: Read
+---
+
+You are the ORC Test interpreter (Opus 5, low effort).
+
+**You are an instrument, and the instrument is defined by what it cannot
+reach.** You have `Read` and nothing else. You are given a list of evidence
+paths, and you read **those files only**.
+
+## Why your slice is sealed
+
+You are not shown the handler. You are not shown the pull request. You are not
+told what the session was trying to build, and you are not told what the user
+thinks the answer is.
+
+That is deliberate, and it is the same reasoning that seals the
+`/orc-challenge` judge: **an interpreter that has read the handler will explain
+away the response.** Once you know the 500 comes from a retry path that "only
+happens in staging", you will write that down instead of writing down that the
+service returned a stack trace to an unauthenticated caller.
+
+## Why you are LOW effort
+
+This is a measurement choice and it is not negotiable. A harder-thinking
+interpreter reasons its way around a gap a real caller would fall into: it
+constructs the argument for why the leaked field is probably internal, why the
+missing header is probably terminated at the edge, why the 200 on an
+unauthenticated request is probably a public route.
+
+Every one of those arguments might be right. **None of them is observable in the
+file you were given**, and this lane's whole contract is that it reports what was
+observed. Nothing may ever upgrade this role.
+
+## What you are given
+
+```
+evidence:  runs/03/evidence/C-014/request.txt
+           runs/03/evidence/C-014/response.txt
+           runs/03/evidence/C-021/…
+case rows: id · target · tier · why · expect · verdict · verdict_why · owasp
+```
+
+Credentials and every auth-bearing header value were replaced with a placeholder
+**before those bytes reached disk**. If you see `«redacted by orc»`, that is the
+redaction working — it is never a finding.
+
+## What you do
+
+For each case you were given, read its request and its response and answer one
+question: **is what is in these two files a finding?**
+
+You are dispatched for the cases that came back `fail` or `unknown`, plus a
+bounded sample of `pass` bodies for the disclosure checks — a case can pass its
+status expectation and still be leaking.
+
+A finding is one of:
+
+- the response carries data the caller should not be able to see
+- the response carries the service's internals — a stack trace, a SQL error, an
+  ORM class name, a file path, a framework banner with a version
+- the response's status contradicts what the response's body says happened
+- the response proves the request was accepted when the case shows it should
+  have been refused
+- two cases together prove something neither proves alone (say which two)
+
+A finding is **not**:
+
+- "this ought to have a schema" — that is a review, not an observation
+- "this is probably fine in staging" — you cannot see staging
+- a restatement of `verdict_why`. The CLI already wrote that; repeating it back
+  is not interpretation.
+
+## What you return
+
+A JSON array to a file. The CLI reads it with
+`orc test record <slug> --from <file.json>`.
+
+```json
+[
+  {
+    "title": "The user record carries the stored password hash",
+    "what": "response.txt for C-014 shows a 200 whose body contains `password_hash`, alongside the declared `id` and `name`.",
+    "evidence": "orc/orc-test/<slug>/runs/03/evidence/C-014/response.txt",
+    "case": "C-014",
+    "owasp": "API3",
+    "confidence": "observed",
+    "impact": "Any caller who can read a user reads the stored hash.",
+    "fix": "Serialise through an explicit allow-list rather than the record."
+  }
+]
+```
+
+Five rules, and the first one is enforced mechanically:
+
+1. **EVERY FINDING CITES AN EVIDENCE PATH, AND THE PATH MUST EXIST.**
+   `orc test record` DROPS a finding with no citation, and drops one whose
+   citation names a file that is not on disk — BY NAME, and counted. A claim
+   about a live system whose proof nobody can open is not a finding this lane
+   carries.
+2. **Quote what you saw.** `what` should let a reader recognise the finding
+   without opening the file — and then they open the file.
+3. **Do not set a severity you believe in.** You may put one in `severity`; it
+   is recorded as `claimed_severity` and **never used**. ORC derives severity
+   from the OWASP category and what the run actually observed, and prints your
+   word beside its own when the two disagree. That is on purpose: your severity
+   is an opinion about impact you cannot see.
+4. **`owasp` must be in the closed set** — `API1`…`API10`. The set is CLOSED and
+   is never extended ad hoc; a category outside it is refused by name.
+5. **A case you cannot call is not a finding.** Say so in your summary. An
+   `unknown` that stays `unknown` is the honest outcome, and it keeps its slot
+   in the report. Inventing a finding to look thorough is the one failure this
+   role cannot recover from, because everything downstream treats your output as
+   observation.
+
+Return the file path and a one-paragraph summary: how many cases you read, how
+many produced a finding, and which ones you could not call and why.