Skip to content

feat: [app] add interceptable command to skip guardrails #346 - #351

Draft
Azamat0315277 wants to merge 6 commits into
developmentfrom
346-app-add-interceptable-command-to-skip-guardrails
Draft

Azamat0315277 wants to merge 6 commits into
developmentfrom
346-app-add-interceptable-command-to-skip-guardrails

Conversation

@Azamat0315277

@Azamat0315277 Azamat0315277 commented May 6, 2026

Copy link
Copy Markdown
Contributor

Applicable issues

Description of changes

Renames skip_out_of_scope_check to cmd_skip_guardrails and migrates it from a static chain input to an interceptable state variable, consistent with the existing CMD_* command pattern.

Changes:

  • Rename env var SKIP_OUT_OF_SCOPE_CHECKCMD_SKIP_GUARDRAILS in DialAppSettings
  • Remove ChainParameters.skip_out_of_scope_check() helper and its SKIP_OUT_OF_SCOPE_CHECK key from ChainParametersConfig
  • Add CMD_SKIP_GUARDRAILS to StateVarsConfig and populate it in init_state (alongside other CMD_* flags)
  • OutOfScopeChecker now reads the flag from state instead of chain inputs
  • Register skip_guardrails as an InterceptableCommand in CommandsInterceptor

Checklist

Status

Draft

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@Azamat0315277 Azamat0315277 self-assigned this May 6, 2026
@Azamat0315277 Azamat0315277 linked an issue May 6, 2026 that may be closed by this pull request
3 tasks
@Azamat0315277 Azamat0315277 changed the title feat: 346 [app] add interceptable command to skip guardrails feat: [app] add interceptable command to skip guardrails #346 May 6, 2026
@Azamat0315277
Azamat0315277 requested a review from navalnica May 6, 2026 07:58
@Azamat0315277
Azamat0315277 force-pushed the 346-app-add-interceptable-command-to-skip-guardrails branch from 697d191 to e87b535 Compare May 6, 2026 11:14
@Azamat0315277

Azamat0315277 commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

/deploy-review

GitHub actions run: 25432347938
Environment URL: review-environment | pipeline

@navalnica

Copy link
Copy Markdown
Contributor

seems good now. I want to ensure ENABLE_DEV_COMMANDS works as expected. will approve once I verify it myself

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[app] add interceptable command to skip guardrails

4 participants