Skip to content

Commit 804e7c4

Browse files
luisleo526claude
andcommitted
ci: tag images by bare commit sha + sync c_abi guardrail
- docker.yml: emit short + long sha tags without `sha-` prefix so every image carries the commit hash directly (previously `sha-abc1234`). - check_c_abi_runtime.py: add `strategy_set_trade_start_time` to the EXPECTED_RUNTIME allowlist; the runtime impl in src/c_abi.cpp landed in 76482e6 but the guardrail was not updated, breaking CI on main. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 84c90eb commit 804e7c4

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

.github/workflows/docker.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,8 @@ jobs:
6262
type=ref,event=pr
6363
type=semver,pattern={{version}}
6464
type=semver,pattern={{major}}.{{minor}}
65-
type=sha,format=short
65+
type=sha,format=short,prefix=
66+
type=sha,format=long,prefix=
6667
type=raw,value=latest,enable={{is_default_branch}}
6768
6869
- name: Build and push

scripts/check_c_abi_runtime.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
#!/usr/bin/env python3
22
"""CI guardrail: runtime-side PF_API implementations in c_abi.cpp stay documented.
33
4-
The header lists ten harness-facing symbols; only ``strategy_set_trace_enabled`` and
5-
``pf_version_get`` are defined in the static runtime — the rest are emitted per-strategy
6-
by the transpiler (see comment in src/c_abi.cpp). If that split changes, update
4+
The header lists eleven harness-facing symbols; only the three in EXPECTED_RUNTIME
5+
are defined in the static runtime — the rest are emitted per-strategy by the
6+
transpiler (see comment in src/c_abi.cpp). If that split changes, update
77
EXPECTED_RUNTIME below and the comment block in c_abi.cpp together.
88
"""
99

@@ -15,7 +15,11 @@
1515

1616
ROOT = Path(__file__).resolve().parents[1]
1717

18-
EXPECTED_RUNTIME = frozenset({"strategy_set_trace_enabled", "pf_version_get"})
18+
EXPECTED_RUNTIME = frozenset({
19+
"strategy_set_trace_enabled",
20+
"strategy_set_trade_start_time",
21+
"pf_version_get",
22+
})
1923

2024
_PF_API_DECL = re.compile(r"^\s*PF_API\b.+\b(\w+)\s*\(")
2125

0 commit comments

Comments
 (0)