@kirjapro/mcp-server 0.19.0 → 0.20.1

package/CHANGELOG.md

@@ -1,5 +1,133 @@
 # Changelog
 
+## [0.20.1] - 2026-10-20
+
+The MCP audit review of 20.10.2026 (N2 item 8, before the OAuth write switch). No tool,
+description or fingerprint changes.
+
+- The core-surface instructions no longer suggest composing a VAT settlement (ALV-tilitys)
+  by hand with `kirjapro_create_transaction`: such a voucher does not mark the VAT period
+  settled, so the period could be settled twice. They redirect it to the Kirjapro app
+  (ALV-yhteenveto → Kirjaa ALV-tilitys), which settles through the same function as
+  `kirjapro_book_vat_settlement` and marks a period settled earlier with a voucher. Purchases
+  and sales may still be composed by hand.
+- `kirjapro_book_vat_settlement` refuses a VAT period that has not ended yet in
+  Europe/Helsinki (`Kausi … ei ole vielä päättynyt, joten sitä ei voi vielä tilittää.`), its
+  dry run included, so VAT booked later in that period is not left unsettled. Migration
+  `20261020180000` refuses it in the database too, for every package version.
+
+## [0.20.0] - 2026-10-20
+
+PR D of the MCP audit of 18.10.2026 (batches 7 to 12 and 14). Against 0.19.0, 3 of the 56
+tool fingerprints change: `kirjapro_book_vat_settlement`, `kirjapro_send_invoice` and
+`kirjapro_send_offer`. The package now bundles its runtime dependencies (no `dependencies`
+in `package.json`).
+
+**PR D release order** (one checklist for every section below; the rollout runbook row 7d0b
+in `docs/guides/ai-workspace-rollout.md` is its source): (1) merge by hand, which deploys the
+web app (until step 2 its Settings page can neither create nor revoke an API key); (2) `npm
+run db:deploy` from clean `main` at low traffic, all six migrations `20261019140000`,
+`20261019150000`, `20261019150100`, `20261019164710`, `20261019170000` and `20261019180000`;
+(3) `npm run functions:deploy` of `dispatch-invoice`, `send-offer-email`, `prh-lookup`,
+`generate-recurring-invoices` and `send-payment-reminders`; (4) `mcp-auth`; (5) `mcp` last;
+(6) this package. For about 11 minutes after step 2 an OAuth connection of a user with MFA
+that was approved just before it may answer 401 once and authorize again.
+
+API key lifecycle and DB-boundary hygiene (MCP audit of 18.10.2026, PR D batch 9). Server
+side only: no tool, description or fingerprint changes, and every published package gets
+the behaviour once migrations `20261019150000` and `20261019150100` and the `mcp-auth`
+deploy are live (steps 2 and 4). `src/database.types.ts` gains `api_keys.membership_id`
+and the RPCs `create_api_key`, `revoke_api_key` and `mcp_api_key_access`.
+
+- A revoked or expired key's live token, or that of a key whose creator left the company,
+  now gets `CREDENTIAL_INVALID` from its next read instead of empty lists ("0 invoices",
+  "not found"): the Data API gate answers such a key token with 401 and a message that
+  starts with the code.
+- `mcp-auth` answers the key of a creator who was removed from the company (even if
+  invited again) with 401, which the server reads as a permanent `CREDENTIAL_INVALID`,
+  instead of the `API_ENTITLEMENT_REQUIRED` 403; a failed check is a transient 503.
+- A credential holds at most 1 000 unresolved idempotency claims. Beyond them a write is
+  refused before it runs (`IDEMPOTENCY_BACKEND_UNAVAILABLE`, the write was not executed).
+- Keys are created and revoked only in the Kirjapro app (Asetukset → MCP-palvelin), which
+  now generates the key in the database and, before revoking, lists the key's writes and
+  offers to pause the recurring invoices with automatic sending that it touched; the revoke
+  button waits for that list. A key stays bound to its creator's membership for good: no
+  later write, not even an operator's, moves or restores the binding.
+
+Untrusted text and send friction (MCP audit of 18.10.2026, PR D batch 7: T2-7, T2-8, T3-40,
+T3-42 to T3-45, T4-13, T4-18; T3-41 was fixed by PR A). Released by the PR D order above
+(the migration `20261019164710` and the Edge functions before this package). Three
+fingerprints change:
+`kirjapro_send_invoice`, `kirjapro_send_offer` and `kirjapro_book_vat_settlement`.
+
+- `kirjapro_send_invoice` and `kirjapro_send_offer` take `dryRun`: nothing is claimed or
+  sent, and the preview names the recipient (wrapped), the channel and the invoice or offer
+  number, with `recipient_confirmation_required`. The results name the recipient(s),
+  wrapped. The preview asks the Edge function with `previewInvoiceId` or `previewOfferId`,
+  never with `invoiceId` or `offerId`, so a function deployed before previews refuses it
+  (`PREVIEW_UNAVAILABLE: … Nothing was sent`) and a dry run can never deliver. The text says
+  `Nothing was sent` only for a refusal or a lost answer.
+- Recipient confirmation (founder decision 19.10.2026): when the customer's e-mail or
+  e-invoice address was created or changed through an MCP credential (an API key or an
+  OAuth connection) in the last 24 hours, the send is refused with a cached
+  `RECIPIENT_CONFIRMATION_REQUIRED` naming the address, unless the call passes
+  `confirmRecipient` equal to it (an e-mail address in any case). A person who saves the
+  customer in the Kirjapro app confirms it too. The rule runs in the Edge functions, so the
+  remote endpoint enforces it as well; signers configured for a contract in the app need no
+  confirmation. The tool descriptions and the server instructions ask the agent to show the
+  address to the user first. An e-invoice refusal also names the operator. The automatic
+  sends have nobody to confirm, so they hold any recipient an MCP credential set that no
+  person has saved since, however old: the daily recurring run leaves the invoice a draft
+  and the template's generation log says why, and the payment reminders skip the customer.
+  An offer without any recipient answers the missing e-mail address, not
+  `RECIPIENT_CONFIRMATION_REQUIRED`.
+- Invoice, reference, offer and customer numbers are no longer trusted as "generated": they
+  stay bare only in the shape Kirjapro allocates (`2026-0001`, digits, `TAR00001`/`SOP00001`,
+  `C00001`) and are wrapped otherwise, in the read tools (also the ALV reconciliation, the
+  offer events and the recurring invoice log) and in the write tools' results and messages
+  (a partial reversal of U-65). The company's own IBAN is bare only with a numeric account
+  part. The server instructions and `kirjapro://security/data-handling` say so.
+- Refusal texts of the bank reconciliation tools and of `kirjapro_book_vat_settlement` that
+  quote stored text (an e-invoice number, a voucher number, the line's currency) are wrapped.
+- An argument that contains `⟦redacted⟧`, which a read result shows in place of text that
+  looked like an instruction, is refused with `INVALID_ARGUMENTS` instead of being stored.
+- `kirjapro_get_received_invoice` wraps `pdf_storage_path` (the sender's file name).
+- `kirjapro_get_recurring_invoice` keeps a `template_data` code bare only when it is a VAT
+  treatment code, VAT category, price-entry mode or account id the server writes.
+- `kirjapro_get_general_ledger` echoes an unknown account wrapped unless it is a number;
+  `kirjapro_book_vat_settlement` takes a 4- or 5-digit `paymentAccount` only.
+- `kirjapro_get_offer` keeps the status codes `from_status` and `to_status` of an offer event
+  bare.
+
+PR D of the MCP audit of 18.10.2026 (batches 8, 11 and 14). No tool definition changes and
+no fingerprint changes. Released by the PR D order above (`20261019170000`, the server-side
+`batch_size` clamp, is in its `db:deploy`; the package does not need it). Before the first
+tag, run the publish workflow once by hand on `main`: its `publish` job now runs in the
+`npm-publish` environment.
+
+- **No runtime dependencies** (T3-58): `@modelcontextprotocol/sdk`, `zod` and
+  `@supabase/supabase-js` are bundled into `dist/index.js` (about 2.7 MB) and are
+  devDependencies; the optional `@supabase/supabase-js` peer dependency is gone. `npx -y
+  @kirjapro/mcp-server` installs this one package, so no unpinned third-party package or
+  install script runs with `KIRJAPRO_API_KEY` in its environment. Up to 0.19.0 about 105
+  packages were resolved at install time.
+- **Stable code for a schema refusal** (T4-16): arguments that fail a tool's input schema
+  answer `INVALID_ARGUMENTS: Input validation error: Invalid arguments for tool …` instead
+  of `MCP error -32602: Input validation error: …`, so every failed call starts with a code.
+  Telemetry already recorded these as `INVALID_ARGUMENTS`.
+- **Telemetry `batch_size`** (T3-60): only a transactions count within 1–250 is sent. A
+  batch call with `transactions: []` or more than 250 items (which the SDK then refuses)
+  made the database refuse the whole telemetry flush, its neighbours' events included.
+- **Release pipeline** (T2-12, T3-78; no package change): `publish-mcp.yml` builds and packs
+  in a job without secrets or publish rights, checks the live types in a job that holds
+  only the read token and installs nothing but the checksum-pinned Supabase CLI 2.111.0
+  release, and publishes the packed tarball from a job
+  in the `npm-publish` environment that installs nothing. `check:release-version` keeps
+  `package.json`, `server.json`, `mcpb/manifest.json` and this file's version heading in
+  step on every pull request; `mcpb:validate` and `mcpb:pack` pin `@anthropic-ai/mcpb@2.1.2`.
+  `check:types-drift` (also in `prepublishOnly`) runs `supabase` from `PATH` and fails unless
+  `supabase --version` is exactly 2.111.0; it no longer runs `npx -y supabase@latest`.
+
 ## [0.19.0] - 2026-10-19
 
 PR C of the MCP audit of 18.10.2026 (batches 3 and 4: posting guards and e-invoice send-state