@crediolabs/policy-synth 0.2.0 → 0.3.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (249) hide show
  1. package/README.md +4 -4
  2. package/dist/adapters/interpreter/adapter.d.ts +19 -25
  3. package/dist/adapters/interpreter/adapter.js +54 -487
  4. package/dist/errors.d.ts +1 -1
  5. package/dist/index.d.ts +1 -4
  6. package/dist/index.js +1 -4
  7. package/dist/install/build-add-context-rule.d.ts +3 -19
  8. package/dist/install/build-add-context-rule.js +3 -98
  9. package/dist/install/build-install-policy.d.ts +4 -53
  10. package/dist/install/build-install-policy.js +44 -123
  11. package/dist/install/index.d.ts +0 -2
  12. package/dist/install/index.js +0 -7
  13. package/dist/predicate/decode.d.ts +1 -1
  14. package/dist/predicate/decode.js +2 -70
  15. package/dist/predicate/encode.js +45 -229
  16. package/dist/predicate/from-json.js +1 -42
  17. package/dist/record/decode.js +3 -8
  18. package/dist/record/freshness.d.ts +17 -11
  19. package/dist/record/freshness.js +28 -18
  20. package/dist/review-card/builder.d.ts +4 -3
  21. package/dist/review-card/builder.js +18 -162
  22. package/dist/review-card/cross-check.js +2 -105
  23. package/dist/review-card/render-leaf.d.ts +4 -0
  24. package/dist/review-card/render-leaf.js +47 -0
  25. package/dist/run/index.d.ts +36 -139
  26. package/dist/run/index.js +125 -437
  27. package/dist/run/schemas.d.ts +168 -1929
  28. package/dist/run/schemas.js +40 -269
  29. package/dist/{synth → simulate}/deny-cases.d.ts +4 -8
  30. package/dist/simulate/deny-cases.js +282 -0
  31. package/dist/simulate/evaluate.d.ts +18 -0
  32. package/dist/simulate/evaluate.js +271 -0
  33. package/dist/simulate/index.d.ts +4 -0
  34. package/dist/simulate/index.js +8 -0
  35. package/dist/synth/compose-from-recording.d.ts +24 -51
  36. package/dist/synth/compose-from-recording.js +120 -282
  37. package/dist/synth/index.d.ts +0 -5
  38. package/dist/synth/index.js +0 -5
  39. package/dist/synth/synthesize-from-recording.d.ts +1 -29
  40. package/dist/synth/synthesize-from-recording.js +41 -356
  41. package/dist/types.d.ts +8 -60
  42. package/dist/types.js +5 -2
  43. package/dist-cjs/adapters/interpreter/adapter.d.ts +19 -25
  44. package/dist-cjs/adapters/interpreter/adapter.js +56 -489
  45. package/dist-cjs/errors.d.ts +1 -1
  46. package/dist-cjs/index.d.ts +1 -4
  47. package/dist-cjs/index.js +1 -4
  48. package/dist-cjs/install/build-add-context-rule.d.ts +3 -19
  49. package/dist-cjs/install/build-add-context-rule.js +1 -97
  50. package/dist-cjs/install/build-install-policy.d.ts +4 -53
  51. package/dist-cjs/install/build-install-policy.js +44 -122
  52. package/dist-cjs/install/index.d.ts +0 -2
  53. package/dist-cjs/install/index.js +2 -23
  54. package/dist-cjs/predicate/decode.d.ts +1 -1
  55. package/dist-cjs/predicate/decode.js +2 -70
  56. package/dist-cjs/predicate/encode.js +45 -229
  57. package/dist-cjs/predicate/from-json.js +1 -42
  58. package/dist-cjs/record/decode.js +3 -8
  59. package/dist-cjs/record/freshness.d.ts +17 -11
  60. package/dist-cjs/record/freshness.js +28 -18
  61. package/dist-cjs/review-card/builder.d.ts +4 -3
  62. package/dist-cjs/review-card/builder.js +23 -167
  63. package/dist-cjs/review-card/cross-check.js +7 -110
  64. package/dist-cjs/review-card/render-leaf.d.ts +4 -0
  65. package/dist-cjs/review-card/render-leaf.js +52 -0
  66. package/dist-cjs/run/index.d.ts +36 -139
  67. package/dist-cjs/run/index.js +125 -444
  68. package/dist-cjs/run/schemas.d.ts +168 -1929
  69. package/dist-cjs/run/schemas.js +41 -270
  70. package/dist-cjs/{synth → simulate}/deny-cases.d.ts +4 -8
  71. package/dist-cjs/simulate/deny-cases.js +286 -0
  72. package/dist-cjs/simulate/evaluate.d.ts +18 -0
  73. package/dist-cjs/simulate/evaluate.js +274 -0
  74. package/dist-cjs/simulate/index.d.ts +4 -0
  75. package/dist-cjs/simulate/index.js +13 -0
  76. package/dist-cjs/synth/compose-from-recording.d.ts +24 -51
  77. package/dist-cjs/synth/compose-from-recording.js +120 -282
  78. package/dist-cjs/synth/index.d.ts +0 -5
  79. package/dist-cjs/synth/index.js +1 -11
  80. package/dist-cjs/synth/synthesize-from-recording.d.ts +1 -29
  81. package/dist-cjs/synth/synthesize-from-recording.js +39 -354
  82. package/dist-cjs/types.d.ts +8 -60
  83. package/dist-cjs/types.js +6 -3
  84. package/package.json +5 -5
  85. package/src/adapters/interpreter/adapter.ts +76 -572
  86. package/src/errors.ts +0 -19
  87. package/src/index.ts +1 -4
  88. package/src/install/build-add-context-rule.ts +5 -139
  89. package/src/install/build-install-policy.ts +93 -214
  90. package/src/install/index.ts +0 -34
  91. package/src/predicate/decode.ts +2 -70
  92. package/src/predicate/encode.ts +49 -261
  93. package/src/predicate/from-json.ts +1 -43
  94. package/src/record/decode.ts +3 -8
  95. package/src/record/freshness.ts +29 -18
  96. package/src/review-card/builder.ts +19 -168
  97. package/src/review-card/cross-check.ts +3 -105
  98. package/src/review-card/render-leaf.ts +48 -0
  99. package/src/run/index.ts +144 -572
  100. package/src/run/schemas.ts +42 -291
  101. package/src/simulate/deny-cases.ts +334 -0
  102. package/src/simulate/evaluate.ts +284 -0
  103. package/src/simulate/index.ts +11 -0
  104. package/src/synth/compose-from-recording.ts +148 -347
  105. package/src/synth/index.ts +0 -13
  106. package/src/synth/synthesize-from-recording.ts +43 -456
  107. package/src/types.ts +13 -49
  108. package/dist/adapters/interpreter/index.d.ts +0 -1
  109. package/dist/adapters/interpreter/index.js +0 -2
  110. package/dist/adapters/oz/adapter.d.ts +0 -20
  111. package/dist/adapters/oz/adapter.js +0 -295
  112. package/dist/adapters/oz/index.d.ts +0 -1
  113. package/dist/adapters/oz/index.js +0 -2
  114. package/dist/codegen/compile-gate.d.ts +0 -33
  115. package/dist/codegen/compile-gate.js +0 -124
  116. package/dist/codegen/index.d.ts +0 -2
  117. package/dist/codegen/index.js +0 -8
  118. package/dist/codegen/template.d.ts +0 -18
  119. package/dist/codegen/template.js +0 -148
  120. package/dist/install/authority-overlap.d.ts +0 -134
  121. package/dist/install/authority-overlap.js +0 -0
  122. package/dist/install/build-install-predicate.d.ts +0 -96
  123. package/dist/install/build-install-predicate.js +0 -444
  124. package/dist/install/build-merge-policy.d.ts +0 -70
  125. package/dist/install/build-merge-policy.js +0 -130
  126. package/dist/install/plan-merge-policy.d.ts +0 -49
  127. package/dist/install/plan-merge-policy.js +0 -86
  128. package/dist/install/read-account-rules.d.ts +0 -100
  129. package/dist/install/read-account-rules.js +0 -283
  130. package/dist/ir/index.d.ts +0 -1
  131. package/dist/ir/index.js +0 -2
  132. package/dist/ir/types.d.ts +0 -140
  133. package/dist/ir/types.js +0 -11
  134. package/dist/mandate/index.d.ts +0 -2
  135. package/dist/mandate/index.js +0 -2
  136. package/dist/mandate/to-ir.d.ts +0 -3
  137. package/dist/mandate/to-ir.js +0 -60
  138. package/dist/mandate/types.d.ts +0 -20
  139. package/dist/mandate/types.js +0 -8
  140. package/dist/seams/index.d.ts +0 -1
  141. package/dist/seams/index.js +0 -2
  142. package/dist/seams/types.d.ts +0 -66
  143. package/dist/seams/types.js +0 -11
  144. package/dist/synth/deny-cases.js +0 -562
  145. package/dist/synth/evaluate.d.ts +0 -39
  146. package/dist/synth/evaluate.js +0 -551
  147. package/dist/synth/harness.d.ts +0 -28
  148. package/dist/synth/harness.js +0 -47
  149. package/dist/synth/minimize.d.ts +0 -4
  150. package/dist/synth/minimize.js +0 -38
  151. package/dist/synth/permit-context.d.ts +0 -15
  152. package/dist/synth/permit-context.js +0 -116
  153. package/dist/synth/predicate-literals.d.ts +0 -5
  154. package/dist/synth/predicate-literals.js +0 -25
  155. package/dist/synth/synthesize-from-mandate.d.ts +0 -20
  156. package/dist/synth/synthesize-from-mandate.js +0 -59
  157. package/dist/verify/envelope.d.ts +0 -15
  158. package/dist/verify/envelope.js +0 -22
  159. package/dist/verify/index.d.ts +0 -3
  160. package/dist/verify/index.js +0 -3
  161. package/dist/verify/simulate.d.ts +0 -31
  162. package/dist/verify/simulate.js +0 -258
  163. package/dist/verify/verify.d.ts +0 -21
  164. package/dist/verify/verify.js +0 -189
  165. package/dist-cjs/adapters/interpreter/index.d.ts +0 -1
  166. package/dist-cjs/adapters/interpreter/index.js +0 -8
  167. package/dist-cjs/adapters/oz/adapter.d.ts +0 -20
  168. package/dist-cjs/adapters/oz/adapter.js +0 -300
  169. package/dist-cjs/adapters/oz/index.d.ts +0 -1
  170. package/dist-cjs/adapters/oz/index.js +0 -8
  171. package/dist-cjs/codegen/compile-gate.d.ts +0 -33
  172. package/dist-cjs/codegen/compile-gate.js +0 -128
  173. package/dist-cjs/codegen/index.d.ts +0 -2
  174. package/dist-cjs/codegen/index.js +0 -14
  175. package/dist-cjs/codegen/template.d.ts +0 -18
  176. package/dist-cjs/codegen/template.js +0 -151
  177. package/dist-cjs/install/authority-overlap.d.ts +0 -134
  178. package/dist-cjs/install/authority-overlap.js +0 -0
  179. package/dist-cjs/install/build-install-predicate.d.ts +0 -96
  180. package/dist-cjs/install/build-install-predicate.js +0 -479
  181. package/dist-cjs/install/build-merge-policy.d.ts +0 -70
  182. package/dist-cjs/install/build-merge-policy.js +0 -134
  183. package/dist-cjs/install/plan-merge-policy.d.ts +0 -49
  184. package/dist-cjs/install/plan-merge-policy.js +0 -90
  185. package/dist-cjs/install/read-account-rules.d.ts +0 -100
  186. package/dist-cjs/install/read-account-rules.js +0 -296
  187. package/dist-cjs/ir/index.d.ts +0 -1
  188. package/dist-cjs/ir/index.js +0 -3
  189. package/dist-cjs/ir/types.d.ts +0 -140
  190. package/dist-cjs/ir/types.js +0 -12
  191. package/dist-cjs/mandate/index.d.ts +0 -2
  192. package/dist-cjs/mandate/index.js +0 -6
  193. package/dist-cjs/mandate/to-ir.d.ts +0 -3
  194. package/dist-cjs/mandate/to-ir.js +0 -63
  195. package/dist-cjs/mandate/types.d.ts +0 -20
  196. package/dist-cjs/mandate/types.js +0 -9
  197. package/dist-cjs/seams/index.d.ts +0 -1
  198. package/dist-cjs/seams/index.js +0 -3
  199. package/dist-cjs/seams/types.d.ts +0 -66
  200. package/dist-cjs/seams/types.js +0 -12
  201. package/dist-cjs/synth/deny-cases.js +0 -568
  202. package/dist-cjs/synth/evaluate.d.ts +0 -39
  203. package/dist-cjs/synth/evaluate.js +0 -554
  204. package/dist-cjs/synth/harness.d.ts +0 -28
  205. package/dist-cjs/synth/harness.js +0 -50
  206. package/dist-cjs/synth/minimize.d.ts +0 -4
  207. package/dist-cjs/synth/minimize.js +0 -41
  208. package/dist-cjs/synth/permit-context.d.ts +0 -15
  209. package/dist-cjs/synth/permit-context.js +0 -119
  210. package/dist-cjs/synth/predicate-literals.d.ts +0 -5
  211. package/dist-cjs/synth/predicate-literals.js +0 -28
  212. package/dist-cjs/synth/synthesize-from-mandate.d.ts +0 -20
  213. package/dist-cjs/synth/synthesize-from-mandate.js +0 -62
  214. package/dist-cjs/verify/envelope.d.ts +0 -15
  215. package/dist-cjs/verify/envelope.js +0 -23
  216. package/dist-cjs/verify/index.d.ts +0 -3
  217. package/dist-cjs/verify/index.js +0 -8
  218. package/dist-cjs/verify/simulate.d.ts +0 -31
  219. package/dist-cjs/verify/simulate.js +0 -261
  220. package/dist-cjs/verify/verify.d.ts +0 -21
  221. package/dist-cjs/verify/verify.js +0 -192
  222. package/src/adapters/interpreter/index.ts +0 -8
  223. package/src/adapters/oz/adapter.ts +0 -376
  224. package/src/adapters/oz/index.ts +0 -9
  225. package/src/codegen/compile-gate.ts +0 -167
  226. package/src/codegen/index.ts +0 -17
  227. package/src/codegen/template.ts +0 -165
  228. package/src/install/authority-overlap.ts +0 -0
  229. package/src/install/build-merge-policy.ts +0 -219
  230. package/src/install/plan-merge-policy.ts +0 -133
  231. package/src/install/read-account-rules.ts +0 -376
  232. package/src/ir/index.ts +0 -13
  233. package/src/ir/types.ts +0 -132
  234. package/src/mandate/index.ts +0 -4
  235. package/src/mandate/to-ir.ts +0 -71
  236. package/src/mandate/types.ts +0 -21
  237. package/src/seams/index.ts +0 -11
  238. package/src/seams/types.ts +0 -81
  239. package/src/synth/deny-cases.ts +0 -663
  240. package/src/synth/evaluate.ts +0 -613
  241. package/src/synth/harness.ts +0 -68
  242. package/src/synth/minimize.ts +0 -48
  243. package/src/synth/permit-context.ts +0 -136
  244. package/src/synth/predicate-literals.ts +0 -27
  245. package/src/synth/synthesize-from-mandate.ts +0 -82
  246. package/src/verify/envelope.ts +0 -28
  247. package/src/verify/index.ts +0 -5
  248. package/src/verify/simulate.ts +0 -311
  249. package/src/verify/verify.ts +0 -243
@@ -39,17 +39,23 @@ export function isBelowThreshold(c) {
39
39
  * under-confidence recording.
40
40
  *
41
41
  * The remediation text branches on which diagnostic bucket is non-empty:
42
- * - unknownContracts only -> user MUST supply an ABI (or re-capture
43
- * against a known protocol version). The contract is real, the
44
- * recorder just cannot decode it.
45
- * - opaqueScVals only -> the recorder encountered a value shape it
46
- * should support but did not decode. The user cannot supply an ABI
47
- * to fix a decoder bug; the guidance explicitly says so and points at
48
- * re-running after a tool upgrade / reporting the path.
49
- * - both -> list both diagnostics AND chain the right remediation for
50
- * each (the user supplies an ABI AND reports the decoder gap).
51
- * - neither (the "denom === 0" path) -> unchanged from the pre-fix
52
- * text; the user did not actually hit a code-level barrier. */
42
+ * - unknownContracts only -> the contract is not in the compiled-in
43
+ * registry. NOT "the contract has no interface": most Soroban
44
+ * contracts publish a typed spec on chain, this package just never
45
+ * reads one. The reason code stays `no-abi` because it is a published
46
+ * enum value, so the TEXT has to carry the correction.
47
+ * - opaqueScVals only -> the recorder met a value shape it should
48
+ * support but did not decode. That is a decoder bug, and the guidance
49
+ * says so rather than implying the caller can fix it.
50
+ * - both -> list both diagnostics and chain the right remediation.
51
+ * - neither (the "denom === 0" path) -> the caller did not actually hit
52
+ * a code-level barrier.
53
+ *
54
+ * Do NOT tell the caller to supply an ABI. There is no input for one:
55
+ * recognition runs off the compiled-in registry plus a known-contract set
56
+ * that no public tool boundary exposes. The old text named an action
57
+ * nobody could take and sent users looking for a file they do not have and
58
+ * would not need. */
53
59
  export function buildLowConfidenceQuestion(c) {
54
60
  const hasUnknown = c.unknownContracts.length > 0;
55
61
  const hasOpaque = c.opaqueScVals.length > 0;
@@ -62,16 +68,20 @@ export function buildLowConfidenceQuestion(c) {
62
68
  }
63
69
  const why = reasons.length === 0 ? 'no diagnostic reason available' : reasons.join('; ');
64
70
  const header = `Recording refused: parseConfidence ${c.overall.toFixed(3)} is below the threshold ${c.thresholdUsed.toFixed(3)}. Diagnostic: ${why}.`;
65
- // The two remediation branches are explicit so the user (or the agent)
66
- // can dispatch on the verb. The "unknown contract" branch is the only
67
- // one that asks for an ABI; the "opaque ScVal" branch asks the user to
68
- // report the path and re-run after a tool upgrade.
71
+ // The remediation branches are explicit so the user (or the agent) can
72
+ // dispatch on the verb. Neither branch asks for an ABI: there is no input
73
+ // that accepts one, and for the unknown-contract branch the contract's
74
+ // own spec is usually on chain already - it is this package that does not
75
+ // read it.
76
+ const unknownAdvice = `The contract is not in this package's built-in registry - "no-abi" means unrecognised here, not that the contract lacks an interface. ` +
77
+ `You cannot supply one: the registry is compiled in. Either build the policy without deriving it from this transaction, or re-capture against a protocol the registry covers. ` +
78
+ `Passing confidenceOverride accepts the recording as it stands, including the arguments this tool could not attribute - do that only if you have checked them yourself.`;
69
79
  if (hasUnknown && hasOpaque) {
70
- return (`${header} Supply an ABI for the unknown contract(s) or re-capture the transaction against a known protocol version; ` +
71
- `also file the opaque ScVal path against the recorder (decoder limitation) and re-run record_transaction after a tool upgrade.`);
80
+ return (`${header} ${unknownAdvice} Separately, file the opaque ScVal path against the recorder (a decoder limitation) ` +
81
+ `and re-run record_transaction after a tool upgrade.`);
72
82
  }
73
83
  if (hasUnknown) {
74
- return `${header} Supply an ABI for the unknown contract(s) or re-capture the transaction against a known protocol version, then re-run record_transaction.`;
84
+ return `${header} ${unknownAdvice}`;
75
85
  }
76
86
  if (hasOpaque) {
77
87
  return `${header} This is a recorder decoder limitation (the value shape is supported in principle but not yet decoded) - the agent cannot fix it by supplying an ABI. Report the opaque ScVal path above against the recorder and re-run record_transaction after a tool upgrade.`;
@@ -1,5 +1,4 @@
1
- import type { ContextRuleDraft, PolicyRef, PredicateNode } from '../types.ts';
2
- import type { SimulationResult } from '../verify/envelope.ts';
1
+ import type { ContextRuleDraft, PredicateNode } from '../types.ts';
3
2
  export interface ReviewCardSummary {
4
3
  ruleName: string;
5
4
  plainEnglish: string;
@@ -17,7 +16,9 @@ export interface ReviewCardSummary {
17
16
  }
18
17
  /** Build a deterministic review-card summary from a policy + context rule +
19
18
  * simulation result. Pure: same inputs -> byte-identical output. */
20
- export declare function buildReviewCardSummary(predicate: PredicateNode | null, policyRefs: PolicyRef[], contextRule: ContextRuleDraft, simulation: SimulationResult): ReviewCardSummary;
19
+ export declare function buildReviewCardSummary(predicate: PredicateNode | null, contextRule: ContextRuleDraft,
20
+ /** Evaluator identity folded into the content hash. */
21
+ backend: 'interpreter-v1' | 'ts-model'): ReviewCardSummary;
21
22
  /** Walk every comparison / membership node of the predicate and invoke
22
23
  * `visit` on each. The walk is depth-first, left-to-right, so the
23
24
  * constraint list is stable across runs. Pure boolean nodes contribute no
@@ -6,38 +6,26 @@
6
6
  // summary being REPRODUCIBLE + TESTABLE + NON-HALLUCINABLE, so this module is
7
7
  // pure: same inputs -> byte-identical output, no clock, no randomness, no I/O.
8
8
  //
9
- // The builder walks two inputs and emits ONE constraint string per leaf or
10
- // primitive, in a fixed deterministic order:
11
- //
12
- // 1. The OZ built-in `PolicyRef`s. Each `spending_limit` primitive becomes a
13
- // `spending_limit(token, limitAmount, windowSecs)` line; other OZ
14
- // primitives (threshold) are skipped - the review card does not quote
15
- // them (they are signer-config concerns, not transactional bounds).
16
- //
17
- // 2. The interpreter `PredicateNode`. One string per constraint leaf,
18
- // rendered by enclosing-comparison kind. Templates (Task 7b):
19
- // - invocation_count_in_window < N -> At most N calls per <window> seconds
20
- // - call_arg[i] in [list] -> Recipient/arg must be one of [list]
21
- // - eq(call_arg[i], literal_vec[...]) -> Path must be exactly [list]
22
- // - oracle_price(asset) OP price -> Only when oracle_price(asset) OP price
23
- // - call_fn == x -> Function must be x
24
- // - call_contract == c -> Contract must be c
25
- // - amount <= v -> Amount <= v
9
+ // The builder walks the interpreter `PredicateNode` and emits ONE constraint
10
+ // string per leaf, in a fixed deterministic order. Templates (Task 7b):
11
+ // - call_arg[i] in [list] -> Recipient/arg must be one of [list]
12
+ // - eq(call_arg[i], literal_vec[...]) -> Path must be exactly [list]
13
+ // - call_fn == x -> Function must be x
14
+ // - call_contract == c -> Contract must be c
15
+ // - amount <= v -> Amount <= v
26
16
  //
27
17
  // The content hash is a stable sha256 hex of a canonical JSON of
28
18
  // { ruleName, plainEnglish, constraints, expiry, backend } - identical
29
19
  // inputs (incl. the context-rule expiry and simulation backend) -> identical
30
20
  // hash. There is no clock; the hash never includes a timestamp.
31
21
  import { createHash } from 'node:crypto';
22
+ import { comparisonOpText, renderHaystackElement, renderVecElement } from "./render-leaf.js";
32
23
  /** Build a deterministic review-card summary from a policy + context rule +
33
24
  * simulation result. Pure: same inputs -> byte-identical output. */
34
- export function buildReviewCardSummary(predicate, policyRefs, contextRule, simulation) {
25
+ export function buildReviewCardSummary(predicate, contextRule,
26
+ /** Evaluator identity folded into the content hash. */
27
+ backend) {
35
28
  const constraints = [];
36
- for (const ref of policyRefs) {
37
- const line = renderOzPrimitive(ref);
38
- if (line !== null)
39
- constraints.push(line);
40
- }
41
29
  if (predicate !== null) {
42
30
  walkPredicate(predicate, (node) => {
43
31
  const line = renderConstraint(node);
@@ -55,7 +43,6 @@ export function buildReviewCardSummary(predicate, policyRefs, contextRule, simul
55
43
  const signerNote = contextRule.signers.length >= 2
56
44
  ? 'any ONE signer may authorise a permitted op under this rule (OZ any-of-N semantic)'
57
45
  : null;
58
- const backend = simulation.backend;
59
46
  const contentHash = computeContentHash({
60
47
  ruleName,
61
48
  plainEnglish,
@@ -66,25 +53,6 @@ export function buildReviewCardSummary(predicate, policyRefs, contextRule, simul
66
53
  });
67
54
  return { ruleName, plainEnglish, constraints, expiry, signerNote, backend, contentHash };
68
55
  }
69
- /** Render the OZ built-in primitive summary line. Only `spending_limit` is
70
- * quoted by the review card (it is the only primitive that defines a
71
- * transactional bound). Other primitives (threshold) are signer-config
72
- * concerns handled by the OZ adapter's own `uncovered` machinery.
73
- * Spending_limit takes `period_ledgers` on-chain (~5s/ledger); the card
74
- * states the window in seconds so the user reads it consistently with the
75
- * interpreter templates. */
76
- function renderOzPrimitive(ref) {
77
- if (ref.kind !== 'oz_builtin')
78
- return null;
79
- const primitive = ref.primitive;
80
- if (primitive.primitive !== 'spending_limit')
81
- return null;
82
- const params = primitive.params;
83
- const limit = params.spending_limit ?? '0';
84
- const periodLedgers = params.period_ledgers ?? 0;
85
- const windowSecs = periodLedgers * 5;
86
- return `spending_limit(${limit}, ${windowSecs})`;
87
- }
88
56
  /** Walk every comparison / membership node of the predicate and invoke
89
57
  * `visit` on each. The walk is depth-first, left-to-right, so the
90
58
  * constraint list is stable across runs. Pure boolean nodes contribute no
@@ -109,21 +77,14 @@ export function describePredicate(predicate) {
109
77
  function walkPredicate(node, visit) {
110
78
  switch (node.op) {
111
79
  case 'and':
112
- case 'or':
113
80
  for (const child of node.children)
114
81
  walkPredicate(child, visit);
115
82
  return;
116
- case 'not':
117
- walkPredicate(node.child, visit);
118
- return;
119
83
  case 'in':
120
84
  visit(node);
121
85
  return;
122
86
  case 'eq':
123
- case 'lt':
124
87
  case 'lte':
125
- case 'gt':
126
- case 'gte':
127
88
  visit(node);
128
89
  return;
129
90
  }
@@ -131,18 +92,13 @@ function walkPredicate(node, visit) {
131
92
  /** Render ONE constraint sentence for ONE interpreter predicate node. The
132
93
  * shape of the output is pinned by Task 7b so the test suite can assert
133
94
  * byte-for-byte equality. Returns `null` when the node is a structural
134
- * boolean (`and` / `or` / `not`) - those are not constraint leaves. */
95
+ * boolean (`and`) - that is not a constraint leaf. */
135
96
  function renderConstraint(node) {
136
97
  switch (node.op) {
137
98
  case 'and':
138
- case 'or':
139
- case 'not':
140
99
  return null;
141
100
  case 'eq':
142
- case 'lt':
143
101
  case 'lte':
144
- case 'gt':
145
- case 'gte':
146
102
  return renderComparison(node);
147
103
  case 'in':
148
104
  return renderMembership(node);
@@ -172,8 +128,8 @@ function renderComparison(node) {
172
128
  }
173
129
  // eq/lte(call_arg_field[i, el, field], <literal>) -> arg[i] element[el].field = <value>
174
130
  // The structured-argument bind pins a scalar field inside a vec element.
175
- // For lt/gt/lte/gte the line reads "OP <value>" so the comparison kind
176
- // is visible; eq reads "= <value>".
131
+ // For lte the line reads "OP <value>" so the comparison kind is visible;
132
+ // eq reads "= <value>".
177
133
  if (left.kind === 'call_arg_field') {
178
134
  const head = `arg[${left.index}] element[${left.element}].${left.field}`;
179
135
  const sep = node.op === 'eq' ? '=' : comparisonOpText(node.op);
@@ -181,10 +137,6 @@ function renderComparison(node) {
181
137
  return `${head} ${sep} ${right.value}`;
182
138
  if (right.kind === 'literal_symbol')
183
139
  return `${head} ${sep} ${right.value}`;
184
- if (right.kind === 'literal_bytes')
185
- return `${head} ${sep} ${right.value}`;
186
- if (right.kind === 'literal_u64')
187
- return `${head} ${sep} ${right.value}`;
188
140
  if (right.kind === 'literal_i128')
189
141
  return `${head} ${sep} ${right.value}`;
190
142
  if (right.kind === 'literal_u32')
@@ -193,22 +145,10 @@ function renderComparison(node) {
193
145
  return `${head} ${sep} [${right.elements.map(renderVecElement).join(', ')}]`;
194
146
  }
195
147
  }
196
- // The bound is compared against the calls ALREADY made in the window, so
197
- // `< N` permits N of them and `<= N` permits one more. Report how many
198
- // calls the rule allows rather than restating the comparison.
199
- if (left.kind === 'invocation_count_in_window' && right.kind === 'literal_u32') {
200
- const allowed = node.op === 'lt' ? right.value : right.value + 1;
201
- return `At most ${allowed} calls per ${left.windowSecs} seconds`;
202
- }
203
- // OP(call_arg[i], <scalar literal>) -> arg[i] OP <value>
204
- //
205
- // The per-call cap. It has to be here because the `amount` template below
206
- // is unreachable for an interpreter policy: `amount` is deliberately not in
207
- // the contract's grammar (dsl.rs - the interpreter sees one authorized
208
- // call, not the transaction's token movements), so a predicate using it is
209
- // refused at install. A bound on the call's own amount ARGUMENT is how a
210
- // cap is actually written, and it was rendering nothing at all - the card
211
- // silently understated the policy.
148
+ // The per-call cap: OP(call_arg[i], <scalar literal>) -> arg[i] OP <value>.
149
+ // The interpreter is passed one authorized call, not the transaction's token
150
+ // movements, so a bound on the call's own amount ARGUMENT is how a cap is
151
+ // written.
212
152
  //
213
153
  // Placed after the literal_vec case above so an exact-sequence `eq` still
214
154
  // reads as a path rather than as a comparison.
@@ -217,28 +157,12 @@ function renderComparison(node) {
217
157
  const sep = node.op === 'eq' ? '=' : comparisonOpText(node.op);
218
158
  if (right.kind === 'literal_i128')
219
159
  return `${head} ${sep} ${right.value}`;
220
- if (right.kind === 'literal_u64')
221
- return `${head} ${sep} ${right.value}`;
222
160
  if (right.kind === 'literal_u32')
223
161
  return `${head} ${sep} ${right.value}`;
224
162
  if (right.kind === 'literal_address')
225
163
  return `${head} ${sep} ${right.value}`;
226
164
  if (right.kind === 'literal_symbol')
227
165
  return `${head} ${sep} ${right.value}`;
228
- if (right.kind === 'literal_bytes')
229
- return `${head} ${sep} ${right.value}`;
230
- // call_arg_scaled is the slippage floor and reads as itself.
231
- if (right.kind === 'call_arg_scaled') {
232
- return `${head} >= arg[${right.index}] * ${right.num}/${right.den}`;
233
- }
234
- }
235
- // amount <= v -> Amount <= v
236
- if (left.kind === 'amount' && right.kind === 'literal_i128') {
237
- return `Amount <= ${right.value}`;
238
- }
239
- // oracle_price(asset) OP price -> Only when oracle_price(asset) OP price
240
- if (left.kind === 'oracle_price' && right.kind === 'oracle_threshold') {
241
- return `Only when oracle_price(${left.asset}) ${comparisonOpText(node.op)} ${right.value} (${right.decimals} dp)`;
242
166
  }
243
167
  // Any other comparison shape is a structural fail-closed: do not surface
244
168
  // a misleading line. Cross-check still requires every leaf produce a
@@ -255,74 +179,6 @@ function renderMembership(node) {
255
179
  }
256
180
  return null;
257
181
  }
258
- function renderVecElement(leaf) {
259
- switch (leaf.kind) {
260
- case 'literal_address':
261
- return leaf.value;
262
- case 'literal_i128':
263
- return leaf.value;
264
- case 'literal_symbol':
265
- return leaf.value;
266
- case 'literal_u32':
267
- return String(leaf.value);
268
- case 'literal_u64':
269
- return leaf.value;
270
- case 'literal_bytes':
271
- return leaf.value;
272
- case 'literal_vec':
273
- return `[${leaf.elements.map(renderVecElement).join(', ')}]`;
274
- case 'call_contract':
275
- case 'call_fn':
276
- case 'call_arg':
277
- case 'call_arg_len':
278
- case 'call_arg_field':
279
- case 'call_arg_scaled':
280
- case 'amount':
281
- case 'window_spent':
282
- case 'now':
283
- case 'valid_until':
284
- case 'invocation_count_in_window':
285
- case 'oracle_price':
286
- return `<${leaf.kind}>`;
287
- // Show the declared basis, not just the digits. A threshold on the wrong
288
- // basis is the one policy error the contract cannot detect, so the review
289
- // card is where a human has to be able to see it.
290
- case 'oracle_threshold':
291
- return `${leaf.value} (${leaf.decimals} dp)`;
292
- }
293
- }
294
- function renderHaystackElement(leaf) {
295
- if (leaf.kind === 'literal_address')
296
- return leaf.value;
297
- if (leaf.kind === 'literal_i128')
298
- return leaf.value;
299
- if (leaf.kind === 'literal_symbol')
300
- return leaf.value;
301
- if (leaf.kind === 'literal_u32')
302
- return String(leaf.value);
303
- if (leaf.kind === 'literal_u64')
304
- return leaf.value;
305
- if (leaf.kind === 'literal_bytes')
306
- return leaf.value;
307
- if (leaf.kind === 'literal_vec') {
308
- return `[${leaf.elements.map(renderHaystackElement).join(', ')}]`;
309
- }
310
- return `<${leaf.kind}>`;
311
- }
312
- function comparisonOpText(op) {
313
- switch (op) {
314
- case 'lt':
315
- return '<';
316
- case 'lte':
317
- return '<=';
318
- case 'gt':
319
- return '>';
320
- case 'gte':
321
- return '>=';
322
- case 'eq':
323
- return '==';
324
- }
325
- }
326
182
  /** Render the plain-English one-liner. Format: `<ruleName>: <constraints>`,
327
183
  * joined by `; ` so the user reads one sentence per constraint. */
328
184
  function renderPlainEnglish(ruleName, constraints) {
@@ -12,6 +12,7 @@
12
12
  // demands the summary carries the corresponding string. A naive summary
13
13
  // that drops a leaf (or fabricates a string not in the predicate) trips the
14
14
  // guard.
15
+ import { comparisonOpText, renderHaystackElement, renderVecElement } from "./render-leaf.js";
15
16
  /** Assert every leaf in the predicate appears as a constraint string in
16
17
  * `summary.constraints`. Returns the missing templates (without the leaf
17
18
  * values filled in) when any leaf was dropped. */
@@ -35,18 +36,11 @@ export function summaryCrossCheck(predicate, summary) {
35
36
  function collect(node, out) {
36
37
  switch (node.op) {
37
38
  case 'and':
38
- case 'or':
39
39
  for (const child of node.children)
40
40
  collect(child, out);
41
41
  return;
42
- case 'not':
43
- collect(node.child, out);
44
- return;
45
42
  case 'eq':
46
- case 'lt':
47
43
  case 'lte':
48
- case 'gt':
49
- case 'gte':
50
44
  pushComparison(node.left, node.right, node.op, out);
51
45
  return;
52
46
  case 'in':
@@ -84,14 +78,6 @@ function pushComparison(left, right, op, out) {
84
78
  out.push(`${head} ${sep} ${right.value}`);
85
79
  return;
86
80
  }
87
- if (right.kind === 'literal_bytes') {
88
- out.push(`${head} ${sep} ${right.value}`);
89
- return;
90
- }
91
- if (right.kind === 'literal_u64') {
92
- out.push(`${head} ${sep} ${right.value}`);
93
- return;
94
- }
95
81
  if (right.kind === 'literal_i128') {
96
82
  out.push(`${head} ${sep} ${right.value}`);
97
83
  return;
@@ -105,13 +91,6 @@ function pushComparison(left, right, op, out) {
105
91
  return;
106
92
  }
107
93
  }
108
- // Mirrors the builder's wording byte-for-byte: how many calls the rule
109
- // permits, since the bound counts the calls already made.
110
- if (left.kind === 'invocation_count_in_window' && right.kind === 'literal_u32') {
111
- const allowed = op === 'lt' ? right.value : right.value + 1;
112
- out.push(`At most ${allowed} calls per ${left.windowSecs} seconds`);
113
- return;
114
- }
115
94
  // Mirrors the builder's per-call cap line. Both sides must render the same
116
95
  // string: cross-check compares the summary against the predicate, so a
117
96
  // shape only ONE of them renders would read as a dropped constraint.
@@ -119,26 +98,12 @@ function pushComparison(left, right, op, out) {
119
98
  const head = `arg[${left.index}]`;
120
99
  const sep = op === 'eq' ? '=' : comparisonOpText(op);
121
100
  if (right.kind === 'literal_i128' ||
122
- right.kind === 'literal_u64' ||
123
101
  right.kind === 'literal_u32' ||
124
102
  right.kind === 'literal_address' ||
125
- right.kind === 'literal_symbol' ||
126
- right.kind === 'literal_bytes') {
103
+ right.kind === 'literal_symbol') {
127
104
  out.push(`${head} ${sep} ${right.value}`);
128
105
  return;
129
106
  }
130
- if (right.kind === 'call_arg_scaled') {
131
- out.push(`${head} >= arg[${right.index}] * ${right.num}/${right.den}`);
132
- return;
133
- }
134
- }
135
- if (left.kind === 'amount' && right.kind === 'literal_i128') {
136
- out.push(`Amount <= ${right.value}`);
137
- return;
138
- }
139
- if (left.kind === 'oracle_price' && right.kind === 'oracle_threshold') {
140
- out.push(`Only when oracle_price(${left.asset}) ${comparisonOpText(op)} ${right.value} (${right.decimals} dp)`);
141
- return;
142
107
  }
143
108
  }
144
109
  function pushMembership(needle, haystack, out) {
@@ -147,71 +112,3 @@ function pushMembership(needle, haystack, out) {
147
112
  const list = haystack.map(renderHaystackElement).join(', ');
148
113
  out.push(`Recipient/arg must be one of [${list}]`);
149
114
  }
150
- function renderVecElement(leaf) {
151
- switch (leaf.kind) {
152
- case 'literal_address':
153
- return leaf.value;
154
- case 'literal_i128':
155
- return leaf.value;
156
- case 'literal_symbol':
157
- return leaf.value;
158
- case 'literal_u32':
159
- return String(leaf.value);
160
- case 'literal_u64':
161
- return leaf.value;
162
- case 'literal_bytes':
163
- return leaf.value;
164
- case 'literal_vec':
165
- return `[${leaf.elements.map(renderVecElement).join(', ')}]`;
166
- case 'call_contract':
167
- case 'call_fn':
168
- case 'call_arg':
169
- case 'call_arg_len':
170
- case 'call_arg_field':
171
- case 'call_arg_scaled':
172
- case 'amount':
173
- case 'window_spent':
174
- case 'now':
175
- case 'valid_until':
176
- case 'invocation_count_in_window':
177
- case 'oracle_price':
178
- return `<${leaf.kind}>`;
179
- // Show the declared basis, not just the digits. A threshold on the wrong
180
- // basis is the one policy error the contract cannot detect, so the review
181
- // card is where a human has to be able to see it.
182
- case 'oracle_threshold':
183
- return `${leaf.value} (${leaf.decimals} dp)`;
184
- }
185
- }
186
- function renderHaystackElement(leaf) {
187
- if (leaf.kind === 'literal_address')
188
- return leaf.value;
189
- if (leaf.kind === 'literal_i128')
190
- return leaf.value;
191
- if (leaf.kind === 'literal_symbol')
192
- return leaf.value;
193
- if (leaf.kind === 'literal_u32')
194
- return String(leaf.value);
195
- if (leaf.kind === 'literal_u64')
196
- return leaf.value;
197
- if (leaf.kind === 'literal_bytes')
198
- return leaf.value;
199
- if (leaf.kind === 'literal_vec') {
200
- return `[${leaf.elements.map(renderHaystackElement).join(', ')}]`;
201
- }
202
- return `<${leaf.kind}>`;
203
- }
204
- function comparisonOpText(op) {
205
- switch (op) {
206
- case 'lt':
207
- return '<';
208
- case 'lte':
209
- return '<=';
210
- case 'gt':
211
- return '>';
212
- case 'gte':
213
- return '>=';
214
- case 'eq':
215
- return '==';
216
- }
217
- }
@@ -0,0 +1,4 @@
1
+ import type { PredicateLeaf } from '../types.ts';
2
+ export declare function renderVecElement(leaf: PredicateLeaf): string;
3
+ export declare function renderHaystackElement(leaf: PredicateLeaf): string;
4
+ export declare function comparisonOpText(op: 'eq' | 'lte'): string;
@@ -0,0 +1,47 @@
1
+ // Leaf and operator rendering shared by the review-card builder and the
2
+ // summary cross-check. Both turn the SAME predicate into human-readable text -
3
+ // the cross-check exists to confirm the builder's summary matches the predicate
4
+ // - so rendering a leaf differently in the two would make the check compare a
5
+ // string against itself under a different spelling and pass on a real mismatch.
6
+ export function renderVecElement(leaf) {
7
+ switch (leaf.kind) {
8
+ case 'literal_address':
9
+ return leaf.value;
10
+ case 'literal_i128':
11
+ return leaf.value;
12
+ case 'literal_symbol':
13
+ return leaf.value;
14
+ case 'literal_u32':
15
+ return String(leaf.value);
16
+ case 'literal_vec':
17
+ return `[${leaf.elements.map(renderVecElement).join(', ')}]`;
18
+ case 'call_contract':
19
+ case 'call_fn':
20
+ case 'call_arg':
21
+ case 'call_arg_len':
22
+ case 'call_arg_field':
23
+ return `<${leaf.kind}>`;
24
+ }
25
+ }
26
+ export function renderHaystackElement(leaf) {
27
+ if (leaf.kind === 'literal_address')
28
+ return leaf.value;
29
+ if (leaf.kind === 'literal_i128')
30
+ return leaf.value;
31
+ if (leaf.kind === 'literal_symbol')
32
+ return leaf.value;
33
+ if (leaf.kind === 'literal_u32')
34
+ return String(leaf.value);
35
+ if (leaf.kind === 'literal_vec') {
36
+ return `[${leaf.elements.map(renderHaystackElement).join(', ')}]`;
37
+ }
38
+ return `<${leaf.kind}>`;
39
+ }
40
+ export function comparisonOpText(op) {
41
+ switch (op) {
42
+ case 'lte':
43
+ return '<=';
44
+ case 'eq':
45
+ return '==';
46
+ }
47
+ }