Commit 0d263c9
## Summary
Makes the flagship **"compile-time shape verification"** claim *true*.
Previously the `@axiom` macro parsed `input::`/`output::` type
annotations but never used them — shape checks were runtime-only, and
the README's `BrokenModel` "Compile error" example could not actually
fire (it also used an aspirational API that doesn't match the code).
This wires genuine verification into the macro. `_verify_axiom_shapes`
runs **during macro expansion** — i.e. at compile time, before the model
is ever constructed or run — tracking the tensor shape from the
`input::` declaration through the layer chain. A provable mismatch
raises an error at expansion:
```
@axiom BrokenModel: compile-time shape mismatch in `output`.
Dense layer expects 128 input feature(s), but the incoming tensor has 256.
Running shape entering this layer: (:batch, 256).
```
## Design — sound but incomplete (by intent)
It rejects **only** mismatches provable from the declared shapes and
literal layer sizes:
- each `Dense`'s declared `in_features` vs the running feature dim along
the chain;
- the computed `output` vs the `output::` declaration.
Anything it cannot statically resolve — `Conv`/pool output geometry,
non-literal layer arguments, unrecognised layers — collapses the running
shape to *unknown* and passes through unchecked. So a **valid model
never receives a false compile error**; only definite, literal-provable
contradictions are rejected.
## Changes
- **`src/dsl/axiom_macro.jl`** — `_verify_axiom_shapes` + helpers
(`_extract_shape_dims`, `_shape_after_layer`, `_flatten_pipeline`, a
shape-preserving-layer whitelist); called at the top of
`generate_axiom_code`.
- **`test/runtests.jl`** — new **"Compile-Time Shape Verification"**
testset. The DSL previously had **zero** `@axiom` tests; this covers
valid chains, the three mismatch types (Dense in-features, first-layer
input, output declaration), the Conv/Flatten soundness case, and the
no-input case.
- **`README.adoc`** — the flagship example rewritten to the **real API**
(`Dense(in,out)`, `(:batch,N)`) showing the actual compile error, with
an explicit soundness caveat.
## Verification
- Full suite: **732/732 passing** (`Pkg.test()`, 5m03s) — includes the
new testset; no regressions.
- The real `@axiom` macro path accepts the
`mnist.jl`/`simple_classifier.jl`-style chains unchanged and rejects the
mismatch cases.
## Follow-up (not in this PR)
The audit's other Tier-1 item — the **PyTorch `.pt`/`.pth` bridge**
claimed as shipped (`scripts/pytorch_to_axiom_descriptor.py` is absent)
— is **not** addressed here: `.pt` files are Python-pickle and importing
them needs PyTorch (Python), which is banned estate-wide. That claim
needs a separate honest correction (the JSON-descriptor path is what
actually ships) or a non-Python reader.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
https://claude.ai/code/session_01UPFC9YQ7g9gc3VnRox42Q1
---
_Generated by [Claude
Code](https://claude.ai/code/session_01UPFC9YQ7g9gc3VnRox42Q1)_
Co-authored-by: Claude <noreply@anthropic.com>
1 parent 59ce4fa commit 0d263c9
3 files changed
Lines changed: 250 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
47 | 51 | | |
48 | 52 | | |
49 | | - | |
50 | | - | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
51 | 60 | | |
| 61 | + | |
52 | 62 | | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
57 | 67 | | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
58 | 71 | | |
59 | 72 | | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
60 | 80 | | |
61 | 81 | | |
62 | 82 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
132 | 132 | | |
133 | 133 | | |
134 | 134 | | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
135 | 296 | | |
136 | 297 | | |
137 | 298 | | |
| |||
140 | 301 | | |
141 | 302 | | |
142 | 303 | | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
143 | 309 | | |
144 | 310 | | |
145 | 311 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
715 | 715 | | |
716 | 716 | | |
717 | 717 | | |
| 718 | + | |
| 719 | + | |
| 720 | + | |
| 721 | + | |
| 722 | + | |
| 723 | + | |
| 724 | + | |
| 725 | + | |
| 726 | + | |
| 727 | + | |
| 728 | + | |
| 729 | + | |
| 730 | + | |
| 731 | + | |
| 732 | + | |
| 733 | + | |
| 734 | + | |
| 735 | + | |
| 736 | + | |
| 737 | + | |
| 738 | + | |
| 739 | + | |
| 740 | + | |
| 741 | + | |
| 742 | + | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
| 749 | + | |
| 750 | + | |
| 751 | + | |
| 752 | + | |
| 753 | + | |
| 754 | + | |
| 755 | + | |
| 756 | + | |
| 757 | + | |
| 758 | + | |
| 759 | + | |
| 760 | + | |
| 761 | + | |
| 762 | + | |
| 763 | + | |
| 764 | + | |
| 765 | + | |
| 766 | + | |
| 767 | + | |
| 768 | + | |
| 769 | + | |
| 770 | + | |
| 771 | + | |
| 772 | + | |
| 773 | + | |
| 774 | + | |
| 775 | + | |
718 | 776 | | |
719 | 777 | | |
720 | 778 | | |
| |||
0 commit comments