Skip to content

fix: resolve structured action context inputs - #229

Open
Optic00 wants to merge 1 commit into
Windshiftapp:mainfrom
Optic00:codex/fix-action-ai-context
Open

fix: resolve structured action context inputs#229
Optic00 wants to merge 1 commit into
Windshiftapp:mainfrom
Optic00:codex/fix-action-ai-context

Conversation

@Optic00

@Optic00 Optic00 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Centralize Action execution-context lookup across templates, ai_extract, and ai_agent.
  • Resolve editor-provided item.* inputs, nested output paths, slices, trigger data, prior values, users, and SCM payloads.
  • Preserve tolerant template placeholders while making missing configured AI inputs fail with a clear error.
  • Serialize structured template and extraction values as JSON.

Behavior note

ai_agent previously skipped unresolved configured inputs and could run with a partial or empty prompt. It now fails the node before starting an LLM call. Present null values remain valid inputs.

Tests

  • go test ./internal/services
  • go test ./... after satisfying the required frontend/dist embed input

This PR includes focused public unit coverage. Test placement is also being discussed in #193; I am happy to adapt it to the maintainers preferred convention.

@stefan-ernst
stefan-ernst self-requested a review August 29, 2026 12:57
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