Skip to content

#64: default codex and inherited automations to astra - #65

Merged
pirog merged 2 commits into
mainfrom
64-enable-astra-default-model
Sep 9, 2026
Merged

#64: default codex and inherited automations to astra#65
pirog merged 2 commits into
mainfrom
64-enable-astra-default-model

Conversation

@pirog

@pirog pirog commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Codex configuration sync would restore gpt-5.6-sol even when the live app had already selected Astra. This changes the repository-owned default to gpt-6-astra, preserves xhigh reasoning, and verifies that all three shipped automations inherit the shared model and reasoning settings.

Related to #64. This delivers the repository changes for the clarified Codex and local OpenClaw scope; local rollout and OpenClaw verification remain pending, so this PR does not close the issue.

Changes:

  • Update the portable shared Codex default and its configuration round-trip assertion.
  • Read the tracked TOML defaults in the shipped-automation test instead of supplying a hardcoded Sol default, and assert Astra inheritance for each automation.
  • Document configuration sync, saved automation reconciliation, existing task selections, and the separate OpenClaw rollout in ADVANCED.md.

Validation at 688ec6bd748afd4d94110092d4f399524346d6ca:

  • Focused configuration and automation checks: 28 passing.
  • bun run test: 125 passing.
  • ESLint, Prettier, and git diff --check: passed.
  • bun run lint:shellcheck: passed after copying the existing local ShellCheck binary into this worktree's ignored dependency directory; the initial lint run could not download that binary under the network restriction.

Public-safe local inspection from September 8, 2026:

  • Generated Codex config already selected Astra at medium, while the tracked source selected Sol at xhigh. Shared/live service-tier settings also differed, so review the full generated-config diff before applying sync.
  • Three saved managed automations explicitly selected Sol at xhigh; the manifest itself has no model or reasoning overrides.
  • Local OpenClaw 2026.7.1-2 selected openai/gpt-5.6-sol through an OpenAI OAuth profile, with no agent model overrides or configured fallback list. Its model listing omitted Astra. That listing is not proof of authenticated account availability.

Pending local rollout:

  • Apply ai:sync, verify a new Codex task without an explicit model selection, and reconcile the saved managed automations through the reviewed automation plan. Existing task selections are independent.
  • Establish installed-version support and authenticated access for openai/gpt-6-astra through the existing OpenClaw provider route. If an upgrade is required, track it as a separate prerequisite. Then allow/select Astra, preserve verified working fallbacks, and verify a fresh session.

No live config sync, automation mutation, model-pin reset, OpenClaw upgrade, or restart was performed. Plugin-cache synchronization, Leia, and the CI-owned build were intentionally skipped because this patch changes no plugin assets, shell behavior, or generated distribution output. EMORI and the unrelated voice cache were left untouched.

@netlify

netlify Bot commented Sep 8, 2026

Copy link
Copy Markdown

Deploy Preview for pirog-boot-me ready!

Name Link
🔨 Latest commit d1ceacb
🔍 Latest deploy log https://app.netlify.com/projects/pirog-boot-me/deploys/6aa053e7cdfe0b000828f6df
😎 Deploy Preview https://deploy-preview-65--pirog-boot-me.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pirog
pirog marked this pull request as ready for review September 9, 2026 13:47
@pirog
pirog merged commit 477d217 into main Sep 9, 2026
13 checks passed
@pirog
pirog deleted the 64-enable-astra-default-model branch September 9, 2026 13:47
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.

1 participant