pineforge-codegen 0.8.0__tar.gz → 0.9.0__tar.gz

{pineforge_codegen-0.8.0 → pineforge_codegen-0.9.0}/CLAUDE.md

@@ -206,14 +206,15 @@ strategy. The taxonomy:
 | Bucket                           | What                                                                 |
 | -------------------------------- | -------------------------------------------------------------------- |
 | `HARD_REJECT_FUNC`               | Calls with no PineForge semantics at all (e.g. `request.financial`). |
-| `HARD_REJECT_NAMESPACE`          | Whole namespaces (e.g. `ticker.*`).                                  |
-| `DIVERGENT_VARS` (warning)       | Built-in variables whose value diverges from TV (e.g. `bar_index`).  |
+| `HARD_REJECT_NAMESPACE`          | Whole-namespace rejects — currently EMPTY (the old `ticker.*` blanket reject became per-function `HARD_REJECT_FUNC` entries: `ticker.{renko,kagi,linebreak,pointfigure,new,modify}`; `ticker.inherit`/`ticker.standard` pass through, `ticker.heikinashi` allowed for the chart's own symbol). |
+| `DIVERGENT_VARS` (warning) / `DIVERGENT_VARS_ERROR` (reject) | Built-in variables whose value diverges from TV (e.g. `bar_index` warns); the ERROR subset (`last_bar_index` — would silently mis-alias to the CURRENT bar index) is rejected because the backtest would be silently wrong. |
 | `BARSTATE_APPROX_VARS` (warning) | Barstate flags PineForge approximates in batch mode.                 |
 | `STRATEGY_UNSUPPORTED_PARAMS`    | Per-strategy.* call kwargs that codegen drops silently.              |
 | `NOT_YET_FUNC`                   | Implementable but currently no codegen — reject loudly.              |
 | `SUPPORTED_*` frozensets         | Per-namespace whitelist of names codegen knows how to emit.          |
 | `varip` VarDecl check            | `varip` declarations rejected outright — batch backtests have no realtime tick state. |
 | TF literal validation            | `request.security` / `request.security_lower_tf` `timeframe` string literals validated against Pine v6 format at parse time. |
+| syminfo na-gap warning           | `SUPPORTED_SYMINFO` = every `SYMINFO_MEMBER_MAP` key, but members whose emission is `na<T>()` or a `get_syminfo_metadata(...)` lookup (root/pricescale/minmove/mincontract/current_contract/expiration_date/isin/sector/industry + fundamentals/recommendations/target_price_*) form `_SYMINFO_SILENT_GAP_FIELDS` (derived from the emission table, so new na-accept fields can't drift out): every read WARNS that the value is na until a data feed injects it. |
 
 
 When extending codegen with a new Pine builtin: add to the corresponding
@@ -230,22 +231,23 @@ These bit us once and are now pinned by `test_compile_smoke.py`'s
 you delete or weaken the special case, the test will tell you.
 
 1. **`year(time)` / `month(time)` / `dayofmonth` / `dayofweek` /
-  `hour(time, tz)` / `minute` / `second` / `weekofyear` function-call
-   form** lowers to an inline lambda that mirrors the engine's
-   `BacktestEngine::_decompose_bar_time()`. The variable form uses
-   `_bar_year()` etc. via `BAR_BUILTINS` (still UTC-only — `_bar_*`
-   accessors call `gmtime_r` unconditionally; chart-tz wiring on the
-   variable side is a separate engine change). The function-call form
-   honors a tz argument: the two-arg form `hour(time, tz)` uses the
-   explicit tz; the one-arg form `hour(time)` defaults to
-   `syminfo_.timezone` (the engine's `SymInfo::timezone` field, "UTC"
-   by default). When the resolved tz is empty / "UTC" / "Etc/UTC" the
-   lambda takes the cheap `gmtime_r` fast path; otherwise it enters a
-   mutex-guarded `setenv("TZ", ...)` + `localtime_r` block that
-   mirrors `pine_tz::ScopedTimezone` (`src/timezone.cpp`, not exposed
-   via any public `<pineforge/...>` header). Harnesses validating
-   against TV exports for non-UTC symbols MUST set chart_tz to match
-   the chart's TZ at export time.
+  `hour(time, tz)` / `minute` / `second` / `weekofyear`** — BOTH the
+   function-call form and the bare variable form lower to the engine's
+   cached, timezone-aware `pine_<field>(ts_ms, tz)` helpers
+   (`session_time.hpp`): the variable form via `BAR_BUILTINS`
+   (`pine_year(current_bar_.timestamp, syminfo_.timezone)` …), the
+   function form via `visit_call.py` (`pine_hour((int64_t)(ts), tz)`),
+   so the two forms agree. The two-arg form uses the explicit tz; the
+   one-arg form defaults to `syminfo_.timezone` (engine
+   `SymInfo::timezone`, "UTC" by default). The old inline
+   `setenv("TZ")+localtime_r` lambda (`tz_time_field_lambda`,
+   codegen/tables.py) is no longer emitted — per-call tzset churn
+   caused a macOS notifyd IPC storm (KI-35) — and has zero call sites.
+   Chart display TZ is a separate engine slot (`chart_timezone_`,
+   `strategy_set_chart_timezone`) intentionally NOT consulted by these
+   builtins; harnesses validating metrics against TV exports for
+   non-UTC charts must still set chart_tz to match the chart's TZ at
+   export time.
 2. **Matrix-returning methods** (`inv` / `pinv` / `transpose` / `copy` /
   `submatrix` / `concat` / `diff` / `mult` / `pow` / `eigenvectors` /
    `kron`) live in `MATRIX_RETURNING_METHODS` (`codegen/tables.py`).
@@ -276,13 +278,21 @@ you delete or weaken the special case, the test will tell you.
    PREVIOUS bar's HLC (`_s_high[1]`, `_s_low[1]`, `_s_close[1]`) per
    Pine v6 semantics with `developing=false`.
 6. **`request.security` is strict.** Only `symbol`, `timeframe`,
-  `expression`, `gaps`, and `lookahead` are allowed. Symbol must resolve
-   to the current chart symbol (`syminfo.tickerid` or `syminfo.ticker`).
-   `gaps` and `lookahead` must be the literal `barmerge.gaps_*` /
+  `expression`, `gaps`, `lookahead`, and `ignore_invalid_symbol` are
+   allowed (`ignore_invalid_symbol` is accepted but inert — the symbol is
+   always the chart symbol, so no symbol can be invalid). Symbol must
+   resolve to the current chart symbol (`syminfo.tickerid` or
+   `syminfo.ticker`, incl. first-binding aliases and
+   `ticker.inherit/standard/heikinashi(<chart sym>)`). `gaps` and
+   `lookahead` must be the literal `barmerge.gaps_*` /
    `barmerge.lookahead_*` member access (codegen does not parse other
-   shapes). `barmerge.lookahead_on` is hard-rejected for backtest-honesty
-   reasons. The `timeframe` argument, when a string literal, is validated
-   against the Pine v6 TF format at parse time (PR #3).
+   shapes). `barmerge.lookahead_on` is ACCEPTED with a repaint WARNING —
+   engine-supported (first-intrabar publication; script-tf publish gating
+   for finer-than-chart targets): see `_check_request_security`'s lookahead
+   branch and `test_request_security_lookahead_on_kwarg_warns`. The old
+   "hard-rejected" wording here was verified stale against the code on
+   2026-07-07 — do not restore it. The `timeframe` argument, when a string
+   literal, is validated against the Pine v6 TF format at parse time (PR #3).
 7. `**SUPPORTED_LOG`** gates `log.{info,warning,error}`. Without it,
   typos like `log.foo("x")` previously emitted a dead empty-string
    statement. Don't remove the gate.

{pineforge_codegen-0.8.0 → pineforge_codegen-0.9.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pineforge-codegen
-Version: 0.8.0
+Version: 0.9.0
 Summary: PineScript v6 to C++ transpiler that targets the pineforge-engine runtime.
 Project-URL: Homepage, https://github.com/pineforge-4pass/pineforge-codegen-oss
 Project-URL: Issues, https://github.com/pineforge-4pass/pineforge-codegen-oss/issues

pineforge_codegen-0.9.0/VERSION

@@ -0,0 +1 @@
+0.9.0