ADR-0001: the kernel / source-adapter / codegen boundary, rewritten from the code (R5 L0) - #255
Merged
Merged
Conversation
luisleo526
force-pushed
the
docs/adr-kernel-adapter-boundary
branch
from
September 19, 2026 23:46
ec77693 to
72c4c4b
Compare
luisleo526
changed the title
luisleo526
force-pushed
the
docs/adr-kernel-adapter-boundary
branch
from
September 20, 2026 00:20
72c4c4b to
08f075d
Compare
The kernel / source-adapter / codegen boundary ADR is rewritten from the code-grounded correction list in docs/design/native-feature-parity.md §2.vi: five trigger kinds and five intents, sizing resolved by the host, the reporting caveat for bare hosts, HTF not reachable from a bare host, no kernel TU reaching a source header, the opaque StrategyOverrides pointer, which TradingView residue actually survives in the kernel, the 57 runtime PF_API symbols and the real C-ABI gap, the two Pine-free examples, the build layout, and the host constructors. The earlier invented adapter priority line is replaced by what order_priority.cpp does. 105 citations, all resolving in this tree. docs/pages/native-engine.md: the host callbacks are not close-only; epochs native_order_v5 / engine_script_run_v17; the CapAttachment / OrderPriority / IntradayCap claim and the rich-run refusal claim are removed; the calculate-on-fill hook, the interim HTF recipe and the open-bar lookahead warning are added. README's architecture section no longer repeats a corrected claim. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CSmYixVAp4NqWpBZxkrLPT
…, kernel twins, bare-host surface, ab9714b defined) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CSmYixVAp4NqWpBZxkrLPT
luisleo526
force-pushed
the
docs/adr-kernel-adapter-boundary
branch
from
September 20, 2026 03:06
08f075d to
39b05f6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
R5 lane L0 (design #256 §2.vi):
docs/adr/0001-kernel-adapter-boundary.mdis rewritten from the code, replacing the two earlier drafts that were written from memory and reviewed as inaccurate.NativeRunSpecscope and its five explicit non-owns, the five trigger kinds and five intents, owners/groups/PointBudget, fills viaon_native_applied+native_events, the reporting caveat for bare hosts.compat/pine/order_priority.cppdoes).PF_APIsymbols and the real C-ABI gap, HTF not reachable from a bare host, what TradingView residue actually survives in the kernel, the build layout (one library, themarket_admission.cppboundary bug fixed by Kernel-only build target: pineforge_kernel + PINEFORGE_BUILD_SOURCE_LAYER (R5 L1) #257), the two Pine-free examples; follow-ups point at the roadmap (Native feature-parity design: gaps, kernel-vs-adapter rulings, lane roadmap #256 §3.2) instead of restating it.Also:
docs/pages/native-engine.mdcorrections (callbacks, epochs v5/v17, removed staleCapAttachment/OrderPriority/IntradayCapand rich-run claims; added the calculate-on-fill hook, the interim HTF recipe, the open-bar lookahead warning) and a README architecture line.Evidence
engine_trade_accessors.cpp:155-167),order_priority.cpp:13-59, andnative_run_spec.hpp:173-174.ci_preflight.pyPASS. Docs only.🤖 Generated with Claude Code
https://claude.ai/code/session_01CSmYixVAp4NqWpBZxkrLPT