Skip to content

added substrate example#386

Merged
peterj merged 2 commits into
kagent-dev:mainfrom
christian-posta:ceposta-substrate-example
Jun 13, 2026
Merged

added substrate example#386
peterj merged 2 commits into
kagent-dev:mainfrom
christian-posta:ceposta-substrate-example

Conversation

@christian-posta

Copy link
Copy Markdown
Collaborator

No description provided.

@christian-posta christian-posta force-pushed the ceposta-substrate-example branch from fc3af7b to d98b553 Compare June 13, 2026 00:00
Signed-off-by: Christian Posta <christian.posta@gmail.com>
@christian-posta christian-posta force-pushed the ceposta-substrate-example branch from d98b553 to 0398cd1 Compare June 13, 2026 00:18
Signed-off-by: Christian Posta <christian.posta@gmail.com>
@christian-posta christian-posta force-pushed the ceposta-substrate-example branch from 96a618e to e8dc22b Compare June 13, 2026 00:41
@christian-posta

Copy link
Copy Markdown
Collaborator Author

Heads up on the deploy check

The deploy job is failing at the Create website preview step, not at the build. The Wrangler action errors with:

In a non-interactive environment, it's necessary to set a CLOUDFLARE_API_TOKEN environment variable for wrangler to work.

That string only appears when secrets.CLOUDFLARE_API_TOKEN resolves to an empty value in the workflow run. The workflow file itself is correct on this branch (npm ci + wrangler deploy --minify, matching the pre-#385 layout). The next build / npm run build:worker step is passing.

This needs a repo-settings fix that I can't make from a PR:

  • Verify CLOUDFLARE_API_TOKEN and CLOUDFLARE_ACCOUNT_ID exist under Settings → Secrets and variables → Actions (not just under an Environment that isn't wired to PR runs).
  • If they're scoped to a GitHub Environment, make sure the deploy job opts into that environment via environment: in .github/workflows/preview.yaml, otherwise PR runs won't see the secrets.
  • Confirm the token still has Workers deploy permissions on the account CLOUDFLARE_ACCOUNT_ID points to.

Nothing in this PR's diff touches CI auth — happy to rebase if any infra fix lands on main.

@peterj peterj merged commit b5c35bd into kagent-dev:main Jun 13, 2026
3 of 4 checks passed
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.

3 participants