Skip to content

chore: harden Varlock runtime contract#311

Closed
WalksWithASwagger wants to merge 1 commit into
mainfrom
codex/issue-310-varlock-runtime-contract
Closed

chore: harden Varlock runtime contract#311
WalksWithASwagger wants to merge 1 commit into
mainfrom
codex/issue-310-varlock-runtime-contract

Conversation

@WalksWithASwagger

Copy link
Copy Markdown
Owner

Summary

  • extend the committed Varlock schema for Replicate and agentic runner settings
  • replace the remaining documented direct .env runtime path with varlock run --inject vars --
  • add agent-safe secret handling and a redacted injection smoke check

Verification

  • npm run docs:check
  • npm run public:check
  • npm run pack:check
  • git diff --check
  • gitleaks dir --redact --no-banner .

Remaining environment check

  • Varlock is not installed in this isolated worktree environment, so varlock load --agent --show-all and the injection smoke check remain for a Varlock-enabled environment.
  • The default Python test environment lacks project dependencies (mcp, jsonschema), so the full test suite was not used as a gate here.

Refs #310

@github-actions

Copy link
Copy Markdown

Agentic PR Review

Verdict: needs-human

Checks:

  • Linked issue via Closes #... or Refs #...: yes
  • Issue acceptance criteria found: 5
  • PR verification section present: yes
  • PR acceptance self-check present: no
  • Diff within v1 limits: yes

This PR is not blocked from human review, but the agentic gate is not satisfied.

@github-actions github-actions Bot added the needs-human Agentic automation stopped for human judgment. label Jul 11, 2026
@WalksWithASwagger

Copy link
Copy Markdown
Owner Author

Superseded by fully verified PR #312, which covers the same issue with passing Varlock audit/injection/staged scan, the full 511-test verification suite, and green GitHub checks.

@WalksWithASwagger WalksWithASwagger deleted the codex/issue-310-varlock-runtime-contract branch July 11, 2026 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-human Agentic automation stopped for human judgment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant