@cspeach/cli 0.9.0 → 1.0.0

package/dist/ui/footer.js

@@ -1,15 +1,22 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import React, { useState, useEffect, useCallback } from 'react';
 import { Box, Text, useInput } from 'ink';
-import TextInput from 'ink-text-input';
+// D29 paste fix (2026-06-12): vendored paste-safe replacement for
+// ink-text-input — multi-chunk pastes were losing every chunk but the
+// last (stale-prop closure in the upstream component). See text-input.tsx.
+import TextInput from './text-input.js';
 import Fuse from 'fuse.js';
 import * as fs from 'node:fs';
 import * as os from 'node:os';
 import * as path from 'node:path';
-const HISTORY_FILE = path.join(os.homedir(), '.cspeach', 'history');
+// Resolved lazily (not at module load) so it honours the HOME/USERPROFILE
+// active at render time — lets tests point it at a temp dir.
+function historyFile() {
+    return path.join(os.homedir(), '.cspeach', 'history');
+}
 function loadHistory() {
     try {
-        return fs.readFileSync(HISTORY_FILE, 'utf8').split('\n').filter(Boolean);
+        return fs.readFileSync(historyFile(), 'utf8').split('\n').filter(Boolean);
     }
     catch {
         return [];
@@ -19,8 +26,9 @@ function appendHistory(line, history) {
     // Skip consecutive duplicates (matches bash HISTCONTROL=ignoredups).
     if (history[history.length - 1] === line)
         return;
-    fs.mkdirSync(path.dirname(HISTORY_FILE), { recursive: true });
-    fs.appendFileSync(HISTORY_FILE, line + '\n');
+    const file = historyFile();
+    fs.mkdirSync(path.dirname(file), { recursive: true });
+    fs.appendFileSync(file, line + '\n');
     history.push(line);
 }
 const PEACH = '#FFCBA4';
@@ -82,7 +90,13 @@ export function Footer({ onSubmit, onSlashTrigger, onHistoryTrigger, onValueChan
             setValue(next === -1 ? '' : history[history.length - 1 - next] ?? '');
             return;
         }
-    });
+        // 2026-06-16 — go dormant while a modal (ask_question / coaching picker)
+        // owns input. Without this, arrow-key navigation INSIDE the modal also
+        // hits this handler (Ink delivers every keystroke to every active
+        // useInput), recalling a history line into the hidden Footer — so the
+        // last command "comes back" in the input once the modal closes. Mirrors
+        // why the TextInput is swapped for a placeholder when disabled.
+    }, { isActive: !disabled });
     const handleSubmit = useCallback((line) => {
         if (!line.trim())
             return;

package/dist/ui/line-resolution.js

@@ -19,8 +19,8 @@
 // pick, so a line flows slash → at → empty-body guard → submit no
 // matter where it entered the pipeline.
 import { isExactSlashMatch, filterSlashEntries } from '../repl/slash-picker.js';
-import { pendingAtTokenPrefix, filterFileChoices } from '../repl/at-picker.js';
-import { SKILL_CATALOG } from '../skill-catalog.js';
+import { pendingAtTokenPrefix } from '../repl/at-picker.js';
+import { SKILL_CATALOG, SKILL_ALIASES } from '../skill-catalog.js';
 export function nextResolutionStep(line, files) {
     const trimmed = line.trim();
     if (trimmed.startsWith('/') && !isExactSlashMatch(trimmed)) {
@@ -38,10 +38,11 @@ export function nextResolutionStep(line, files) {
     const filenames = files.map((f) => f.filename);
     const atPrefix = pendingAtTokenPrefix(trimmed, filenames);
     if (atPrefix !== null && files.length > 0) {
-        const seeded = filterFileChoices(atPrefix, files);
-        if (seeded.length === 1) {
-            return emptyBodyGuard(trimmed.replace(/@\S*$/, `@${seeded[0].filename}`));
-        }
+        // Always open the picker when the line ends in an unresolved @-token —
+        // even when it narrows to a single file. The old single-match shortcut
+        // silently auto-resolved AND auto-submitted, so the user never saw which
+        // file was attached and couldn't keep typing (e.g. add `--scope`). Showing
+        // the palette + inserting-on-pick (App handles that) is the correct flow.
         return { kind: 'open-at', prefix: atPrefix, line: trimmed };
     }
     // Empty workspace: nothing to pick — let the downstream @-resolution
@@ -68,6 +69,16 @@ function emptyBodyGuard(line) {
         const splitIdx = line.search(/\s/);
         const head = splitIdx === -1 ? line : line.slice(0, splitIdx);
         const body = splitIdx === -1 ? '' : line.slice(splitIdx).trim();
+        // Retired-but-redirected name with no body: prefill the NEW name so the
+        // user both learns the rename and doesn't dispatch an empty prompt.
+        const aliasTarget = SKILL_ALIASES[head.toLowerCase().slice(1)];
+        if (aliasTarget !== undefined && body.length === 0) {
+            return {
+                kind: 'prefill',
+                text: `/${aliasTarget} `,
+                hint: `↪ ${head.toLowerCase()} is now /${aliasTarget} — type your prompt and press Enter`,
+            };
+        }
         const isSkill = SKILL_CATALOG.some((s) => `/${s.name}` === head.toLowerCase());
         if (isSkill && body.length === 0) {
             return {

package/dist/ui/session-timeline.js

@@ -6,6 +6,7 @@ const MUTATING_TOOLS = new Set([
     'sap_set_source', 'sap_update_method', 'sap_delete_object', 'sap_activate',
     'sap_create_object', 'sap_transport_release', 'sap_message_maintain',
     'sap_number_range_intervals', 'sap_service_binding_publish',
+    'sap_service_binding_unpublish',
 ]);
 export function toTimelineEntry(call) {
     // args shape depends on the tool; extract defensively.

package/dist/ui/text-input.js

@@ -0,0 +1,150 @@
+import { jsx as _jsx } from "react/jsx-runtime";
+/**
+ * PasteSafeTextInput — drop-in replacement for `ink-text-input` (D29,
+ * "chewed words" during seed-session paste; 2026-06-12).
+ *
+ * ROOT CAUSE of the chewing, verified by repro test: Ink's App drains ALL
+ * pending stdin chunks in one synchronous loop (components/App.js
+ * `handleReadable`: `while ((chunk = stdin.read()) !== null) emit('input')`),
+ * and a large terminal paste arrives as MULTIPLE chunks. ink-text-input
+ * computes `nextValue = value.slice(0, cursor) + input + …` from the
+ * `value` PROP of the last committed render — a stale closure for every
+ * chunk after the first in the same tick. With chunks A then B, B's
+ * handler still sees the pre-A value, so A is silently dropped:
+ * `stdin.write('hello '); stdin.write('world')` → value `"world"`.
+ *
+ * FIX: mirror value + cursor in refs that are updated SYNCHRONOUSLY inside
+ * the input handler, so consecutive same-tick chunks compound. The refs
+ * re-sync from props on commit, so external value pushes (submit-clear,
+ * palette prefill) still win between keystrokes. This also fixes the
+ * sibling hazard: an Enter that lands in the same drain as the paste tail
+ * now submits the COMPLETE value instead of the stale prop.
+ *
+ * Rendering, cursor drawing, placeholder, and mask behaviour are vendored
+ * verbatim from ink-text-input v6 (MIT, © Vadim Demedes) minus
+ * `highlightPastedText` (Footer never used it).
+ */
+import { useState, useEffect, useRef } from 'react';
+import { Text, useInput } from 'ink';
+import chalk from 'chalk';
+export function PasteSafeTextInput({ value: originalValue, placeholder = '', focus = true, mask, showCursor = true, onChange, onSubmit, }) {
+    const [cursorOffset, setCursorOffset] = useState((originalValue || '').length);
+    // The paste-safety core: synchronous mirrors of value + cursor. The
+    // input handler reads/writes THESE, never the (possibly stale) props.
+    const valueRef = useRef(originalValue);
+    const cursorRef = useRef(cursorOffset);
+    // Values we've sent through onChange whose prop echo hasn't been
+    // observed yet. Needed because passive effects flush LATE: with N
+    // same-tick chunks, the echo effect for chunk k can run AFTER the
+    // handler for chunk k+1, and blindly assigning `valueRef.current =
+    // originalValue` would rewind the mirror to a stale value (observed in
+    // the repro as words landing out of order). An echo — even a stale
+    // one — must never overwrite the mirror; only EXTERNAL pushes may.
+    const pendingEchoes = useRef([]);
+    // Re-sync the mirrors on EXTERNAL value pushes only (parent setValue('')
+    // after submit, palette prefill). Echoes of our own onChange — current
+    // or stale — are recognised via pendingEchoes and skipped: the mirror
+    // is already at or ahead of them.
+    useEffect(() => {
+        const echoIdx = pendingEchoes.current.indexOf(originalValue);
+        if (echoIdx !== -1) {
+            pendingEchoes.current.splice(0, echoIdx + 1);
+            return;
+        }
+        pendingEchoes.current = [];
+        valueRef.current = originalValue;
+        if (cursorRef.current > originalValue.length) {
+            cursorRef.current = originalValue.length;
+            setCursorOffset(originalValue.length);
+        }
+    }, [originalValue]);
+    // Clamp rendered cursor when the value shrinks under it (vendored
+    // semantic from ink-text-input's first effect).
+    useEffect(() => {
+        if (!focus || !showCursor)
+            return;
+        setCursorOffset((previous) => previous > originalValue.length ? originalValue.length : previous);
+    }, [originalValue, focus, showCursor]);
+    const value = mask ? mask.repeat(originalValue.length) : originalValue;
+    let renderedValue = value;
+    let renderedPlaceholder = placeholder ? chalk.grey(placeholder) : undefined;
+    // Fake cursor — vendored rendering from ink-text-input v6.
+    if (showCursor && focus) {
+        renderedPlaceholder =
+            placeholder.length > 0
+                ? chalk.inverse(placeholder[0]) + chalk.grey(placeholder.slice(1))
+                : chalk.inverse(' ');
+        renderedValue = value.length > 0 ? '' : chalk.inverse(' ');
+        let i = 0;
+        for (const char of value) {
+            renderedValue += i === cursorOffset ? chalk.inverse(char) : char;
+            i++;
+        }
+        if (value.length > 0 && cursorOffset === value.length) {
+            renderedValue += chalk.inverse(' ');
+        }
+    }
+    useInput((input, key) => {
+        if (key.upArrow ||
+            key.downArrow ||
+            (key.ctrl && input === 'c') ||
+            key.tab ||
+            (key.shift && key.tab)) {
+            return;
+        }
+        if (key.return) {
+            // valueRef (not the prop) — so an Enter in the same stdin drain as
+            // the paste tail submits the complete line.
+            if (onSubmit)
+                onSubmit(valueRef.current);
+            return;
+        }
+        const currentValue = valueRef.current;
+        let nextCursorOffset = cursorRef.current;
+        let nextValue = currentValue;
+        if (key.leftArrow) {
+            if (showCursor)
+                nextCursorOffset--;
+        }
+        else if (key.rightArrow) {
+            if (showCursor)
+                nextCursorOffset++;
+        }
+        else if (key.backspace || key.delete) {
+            if (cursorRef.current > 0) {
+                nextValue =
+                    currentValue.slice(0, cursorRef.current - 1) +
+                        currentValue.slice(cursorRef.current, currentValue.length);
+                nextCursorOffset--;
+            }
+        }
+        else {
+            nextValue =
+                currentValue.slice(0, cursorRef.current) +
+                    input +
+                    currentValue.slice(cursorRef.current, currentValue.length);
+            nextCursorOffset += input.length;
+        }
+        if (nextCursorOffset < 0)
+            nextCursorOffset = 0;
+        if (nextCursorOffset > nextValue.length)
+            nextCursorOffset = nextValue.length;
+        // Synchronous mirror update FIRST — the next same-tick chunk builds
+        // on this, not on the stale render.
+        valueRef.current = nextValue;
+        cursorRef.current = nextCursorOffset;
+        setCursorOffset(nextCursorOffset);
+        if (nextValue !== currentValue) {
+            // Register the outgoing value so its (possibly late) prop echo is
+            // recognised and never rewinds the mirror.
+            pendingEchoes.current.push(nextValue);
+            onChange(nextValue);
+        }
+    }, { isActive: focus });
+    return (_jsx(Text, { children: placeholder
+            ? value.length > 0
+                ? renderedValue
+                : renderedPlaceholder
+            : renderedValue }));
+}
+export default PasteSafeTextInput;

package/dist/ui/widgets/ask-question-modal.js

@@ -17,7 +17,10 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
  */
 import { useState, useEffect } from 'react';
 import { Box, Text, useInput } from 'ink';
-import TextInput from 'ink-text-input';
+// D29 paste fix (2026-06-12): paste-safe replacement for ink-text-input —
+// pasted free-form answers were losing every chunk but the last on large
+// pastes (stale-prop closure upstream). See ../text-input.tsx.
+import TextInput from '../text-input.js';
 const PEACH = '#FFCBA4';
 /**
  * Reclaim raw-mode stdin when an Ink modal mounts (2026-06-08).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cspeach/cli",
-  "version": "0.9.0",
+  "version": "1.0.0",
   "description": "CSPeach — AI-assisted ABAP development CLI. Direct SAP access, 34 skills, safety gates.",
   "keywords": [
     "abap",
@@ -41,42 +41,58 @@
     "smoke:all": "pnpm zcspeach:smoke && pnpm adt:smoke",
     "zcspeach:smoke": "tsx scripts/zcspeach-smoke.ts",
     "bench:models": "tsx scripts/bench-models.ts",
-    "prepare": "husky"
+    "fiori": "tsx src/tools/fiori/bin.ts",
+    "prepare": "husky",
+    "prepublishOnly": "node -e \"const d=(require('./package.json').dependencies||{})['@cspeach/sap-client']||''; if(d.startsWith('file:')||d.startsWith('link:')){console.error('publish blocked: @cspeach/sap-client is a local link ('+d+') — point it at the published version before npm publish'); process.exit(1)}\""
   },
   "engines": {
     "node": ">=20.0.0"
   },
   "dependencies": {
     "@anthropic-ai/sdk": "^0.90.0",
-    "@cspeach/sap-client": "^0.1.3",
+    "@cspeach/sap-client": "^0.1.10",
     "@iarna/toml": "^2.2.5",
     "@inkjs/ui": "^2.0.0",
     "@inquirer/prompts": "^7.0.0",
     "@noble/ed25519": "2.1.0",
     "@noble/hashes": "1.4.0",
+    "@sap-ux/fiori-elements-writer": "^3.0.39",
+    "@sap-ux/fiori-freestyle-writer": "3.0.12",
     "chalk": "^5.3.0",
     "cli-highlight": "^2.1.11",
     "cli-table3": "^0.6.5",
     "clipboardy": "^5.3.1",
+    "cross-spawn": "^7.0.6",
     "diff": "^9.0.0",
+    "ejs": "3.1.10",
     "fuse.js": "^7.0.0",
     "ink": "^5.0.0",
     "ink-text-input": "^6.0.0",
     "jose": "^5.9.0",
     "keytar": "^7.9.0",
+    "mammoth": "^1.8.0",
     "marked": "^11.2.0",
     "marked-terminal": "^6.2.0",
+    "mem-fs": "2.1.0",
+    "mem-fs-editor": "9.4.0",
     "node-machine-id": "^1.1.12",
     "openai": "4.67.3",
     "react": "^18.3.0",
+    "turndown": "^7.2.0",
+    "unpdf": "^0.12.1",
     "zod": "^3.23.0"
   },
   "devDependencies": {
+    "@types/cross-spawn": "^6.0.6",
     "@types/diff": "^7.0.2",
+    "@types/ejs": "^3.1.5",
     "@types/node": "^20.17.0",
     "@types/react": "^18.3.0",
+    "@types/turndown": "^5.0.5",
+    "docx": "^9.0.2",
     "husky": "^9.1.7",
     "ink-testing-library": "^4.0.0",
+    "pdf-lib": "^1.17.1",
     "tsx": "^4.19.0",
     "typescript": "^5.7.0",
     "vitest": "^2.1.0"

package/bench/README.md

@@ -1,78 +0,0 @@
-# Model bench
-
-Side-by-side outputs from Opus, Sonnet, and Haiku on the same ABAP prompt so
-you can decide per-skill which model carries the load before changing any
-default.
-
-## Run
-
-```bash
-pnpm bench:models
-pnpm bench:models --prompt abap-explain-bdef-handler
-pnpm bench:models --models claude-opus-4-7,claude-sonnet-4-6
-```
-
-Routes through whichever LLM mode `cspeach config` is in — managed (proxy),
-BYOK, AI Hub, or local. In managed mode the proxy injects the **real skill
-body from KV** as the system prompt, so each bench call is end-to-end
-identical to what a real `/abap-explain` invocation would do — just on a
-different model.
-
-## Prompts
-
-Drop new prompts in `prompts/<name>.md`. Format:
-
-```markdown
----
-skill: abap-explain          # required — must be a real CSPeach skill name
-description: short label
----
-
-## user
-[the user-side prompt the model receives]
-```
-
-Optionally include a `## system` section — it's **only** used in non-managed
-modes (BYOK, AI Hub, local) where there's no proxy to inject a skill body.
-In managed mode `## system` is ignored.
-
-## Results
-
-`results/<prompt-name>/<model>.md` — full assistant text plus latency, tokens,
-and a USD estimate (using the same pricing table the cost footer uses).
-
-Read both files side by side. If Sonnet looks like a write-off on a given
-skill, keep the skill on Opus. If it looks "as good or better," that skill
-graduates down on the next config refactor.
-
-## Cost note
-
-Each prompt × model call is one full skill turn through the proxy. The proxy
-loads the full skill body as system prompt, so input is in the 5-15k token
-range. A full 3-prompt × 3-model bench is roughly $0.30–$0.80 depending on
-output verbosity. (Haiku columns will be much cheaper than Opus.)
-
-## Verdict log
-
-Findings from each bench run, oldest first. Each entry should name the run
-date, the prompts that were benched, and the model decision per skill — so
-the config refactor (`resolveModelForSkill` + `[models.skills]`) can be
-driven from evidence, not guesses.
-
-### 2026-05-18 — initial 3-prompt run
-
-Prompts: `abap-explain-bdef-handler`, `abap-document-cds`, `abap-test-method`.
-
-| Skill | Sonnet 4.6 | Haiku 4.5 | Decision |
-|---|---|---|---|
-| `abap-explain`  | matches Opus, catches `IN LOCAL MODE` non-retrigger nuance | typo `cl_abap_behav_handler` (wrong class name) | **Sonnet OK; Haiku unsafe** |
-| `abap-document` | matches Opus + adds client-handling + `@VDM.viewType` value | invented `DSOD` authority object; misread fuzzinessThreshold direction | **Sonnet OK; Haiku unsafe** |
-| `abap-test`     | comparable rigor; asserts exception **textid** (Opus only `assert_bound`) | clean code; coverage-estimate prose shaky | **Sonnet OK; Haiku borderline** |
-
-Pattern: SAP/ABAP corpus is thinner than Python/JS — Haiku hallucinates SAP
-terms on multi-paragraph synthesis. Sonnet does not. Do NOT route any
-ABAP-content skill to Haiku based on this evidence.
-
-Skills NOT yet benched (kept on Opus by default): `abap-generate`, `-rap`,
-`-modernize`, `-design`, `-refactor`, `-review`, `-cca`, anything else
-producing new code or architectural decisions. Bench before routing.

package/bench/prompts/abap-document-cds.md

@@ -1,44 +0,0 @@
----
-skill: abap-document
-description: Document a CDS interface view with associations + currency annotation
----
-
-## user
-
-Please document this CDS view:
-
-```cds
-@AccessControl.authorizationCheck: #CHECK
-@EndUserText.label: 'Sales Order — Interface View'
-@Search.searchable: true
-define view entity ZI_SalesOrder
-  as select from zsales_order as Header
-
-  association [0..1] to ZI_Customer       as _Customer       on $projection.CustomerID = _Customer.CustomerID
-  association [0..*] to ZI_SalesOrderItem as _SalesOrderItem on $projection.SalesOrderID = _SalesOrderItem.SalesOrderID
-  association [0..1] to I_Currency        as _Currency       on $projection.CurrencyCode = _Currency.Currency
-
-{
-  key Header.sales_order_id        as SalesOrderID,
-      Header.customer_id           as CustomerID,
-      Header.order_date            as OrderDate,
-
-      @Search.defaultSearchElement: true
-      @Search.fuzzinessThreshold: 0.8
-      @ObjectModel.text.element: [ '_Customer.CustomerName' ]
-      Header.customer_id           as CustomerIDForSearch,
-
-      @Semantics.amount.currencyCode: 'CurrencyCode'
-      Header.net_amount            as NetAmount,
-      Header.currency_code         as CurrencyCode,
-
-      Header.created_at            as CreatedAt,
-      Header.created_by            as CreatedBy,
-      Header.last_changed_at       as LastChangedAt,
-      Header.last_changed_by       as LastChangedBy,
-
-      _Customer,
-      _SalesOrderItem,
-      _Currency
-}
-```

package/bench/prompts/abap-explain-bdef-handler.md

@@ -1,57 +0,0 @@
----
-skill: abap-explain
-description: Explain a managed RAP behavior pool with a validation + determination
----
-
-## user
-
-Please explain this behavior pool:
-
-```abap
-CLASS lhc_salesorder DEFINITION INHERITING FROM cl_abap_behavior_handler.
-  PRIVATE SECTION.
-    METHODS validateorderdate FOR VALIDATE ON SAVE
-      IMPORTING keys FOR SalesOrder~validateorderdate.
-    METHODS determinenetamount FOR DETERMINE ON MODIFY
-      IMPORTING keys FOR SalesOrder~determinenetamount.
-ENDCLASS.
-
-CLASS lhc_salesorder IMPLEMENTATION.
-  METHOD validateorderdate.
-    READ ENTITIES OF zr_salesorder IN LOCAL MODE
-      ENTITY SalesOrder FIELDS ( OrderDate ) WITH CORRESPONDING #( keys )
-      RESULT DATA(lt_orders).
-
-    LOOP AT lt_orders INTO DATA(ls_order).
-      IF ls_order-OrderDate < cl_abap_context_info=>get_system_date( ).
-        APPEND VALUE #( %tky = ls_order-%tky ) TO failed-salesorder.
-        APPEND VALUE #(
-          %tky = ls_order-%tky
-          %msg = new_message_with_text(
-                   severity = if_abap_behv_message=>severity-error
-                   text     = 'Order date must be today or in the future' )
-        ) TO reported-salesorder.
-      ENDIF.
-    ENDLOOP.
-  ENDMETHOD.
-
-  METHOD determinenetamount.
-    READ ENTITIES OF zr_salesorder IN LOCAL MODE
-      ENTITY SalesOrder FIELDS ( Quantity UnitPrice ) WITH CORRESPONDING #( keys )
-      RESULT DATA(lt_orders).
-
-    DATA lt_update TYPE TABLE FOR UPDATE zr_salesorder\\SalesOrder.
-    LOOP AT lt_orders INTO DATA(ls_order).
-      APPEND VALUE #(
-        %tky      = ls_order-%tky
-        NetAmount = ls_order-Quantity * ls_order-UnitPrice
-        %control-NetAmount = if_abap_behv=>mk-on
-      ) TO lt_update.
-    ENDLOOP.
-
-    MODIFY ENTITIES OF zr_salesorder IN LOCAL MODE
-      ENTITY SalesOrder UPDATE FIELDS ( NetAmount )
-        WITH lt_update.
-  ENDMETHOD.
-ENDCLASS.
-```

package/bench/prompts/abap-test-method.md

@@ -1,42 +0,0 @@
----
-skill: abap-test
-description: Generate an ABAP Unit test class for a small business rule method
----
-
-## user
-
-Please write an ABAP Unit test class for this method:
-
-```abap
-CLASS zcl_discount_calculator DEFINITION
-  PUBLIC FINAL CREATE PUBLIC.
-
-  PUBLIC SECTION.
-    "! Returns the discount percentage (0..30) for a given net amount.
-    "!   < 1000  → 0%
-    "!   < 5000  → 5%
-    "!   < 10000 → 10%
-    "!   >= 10000 → 15%
-    "! Raises zcx_discount_error on negative input.
-    METHODS calculate_discount
-      IMPORTING iv_net_amount      TYPE p LENGTH 13 DECIMALS 2
-      RETURNING VALUE(rv_discount) TYPE p LENGTH 3  DECIMALS 2
-      RAISING   zcx_discount_error.
-ENDCLASS.
-
-CLASS zcl_discount_calculator IMPLEMENTATION.
-  METHOD calculate_discount.
-    IF iv_net_amount < 0.
-      RAISE EXCEPTION TYPE zcx_discount_error
-        EXPORTING textid = zcx_discount_error=>negative_input.
-    ENDIF.
-
-    rv_discount = COND #(
-      WHEN iv_net_amount <  1000 THEN '0.00'
-      WHEN iv_net_amount <  5000 THEN '5.00'
-      WHEN iv_net_amount < 10000 THEN '10.00'
-      ELSE                            '15.00'
-    ).
-  ENDMETHOD.
-ENDCLASS.
-```