Skip to content

local:check smoke checks predate the v1 object plugin export #338

Closed as not planned

Description

@terisuke

Since 2026-09-04 (guardrail.ts switched to export default { id, server } v1 form), three embedded smoke checks in scripts/local-dev-deploy.sh call mod.default(...) as a function and fail with "default is an instance of Object": aggregate policy fire, team plugin loads, team fallback. Pre-existing on dev @ 01516d1 — reproduced 2026-09-12 during the deepseek-flash deploy verification. The checks need the v1-aware call shape (mod.default.server(input, opts)) like plugin-loads.test.ts uses.

Activity

  1. terisuke commented on Oct 1, 2026

    @terisuke
    Author

    2026-10-01の作業でも同じ3件の失敗を確認しました。重複Issueは作らず本件に集約します。

    bun run local:check:

    • guardrails aggregate plugin fires review/git/UAT policy hooks: mod.default(...) が Object で TypeError
    • guardrails team plugin loads team/background/team_status: 同じ呼び出し形の TypeError
    • guardrails team fallback uses zai-coding-plan/glm-5.3: 同じ呼び出し形の TypeError

    v1 exportを正しく呼ぶ実装・loader経路との照合が必要です。単に失敗チェックを除去するのではなく、
    各smokeが意図する挙動をv1形式で実際に検査し、修正前RED→修正後GREENを残すのが受け入れ条件です。
    このコメント時点では修正していません。モデルカタログ不一致は #339 の別件です。

  2. terisuke commented on Oct 19, 2026

    @terisuke
    Author

    2026-10-19 の 2.x 再プラットフォーム(#343)で、scripts/local-dev-deploy.sh は持ち越していません(1.x の dist バイナリと 1.x のプラグインファイル配置に依存していたため)。この Issue の前提のスモークチェックは新しい dev に存在しないので閉じます。2.x での読み込み確認は packages/guardrails/profile/src/plugin-loads.test.ts(上流 loader の 3 つの事実を固定)と src/v2/adapter.test.ts が担います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtOut-of-scope problems tracked for later

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions