Skip to content

Add variable-aware conversion runtime and refresh AI Shortcuts UI - #1

Open
midasavocado wants to merge 1 commit into
mainfrom
codex/refactor-conversions.js-for-variable-handling
Open

midasavocado wants to merge 1 commit into
mainfrom
codex/refactor-conversions.js-for-variable-handling

Conversation

@midasavocado

Copy link
Copy Markdown
Owner

Summary

  • introduce a ConversionRuntime to normalize params, handle $/! variable markers, and auto-inject WFTextToken attachments before substitution
  • update converters to share runtime context and support UUID naming while keeping existing templates
  • refresh the AI Shortcuts interface with hero highlights, features grid, workflow steps, and sample prompt chips

Testing

  • not run (not available)

https://chatgpt.com/codex/tasks/task_e_68f92a02c504832e980eeee7f864a837

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for enochfiles canceled.

Name Link
🔨 Latest commit ae0c0f8
🔍 Latest deploy log https://app.netlify.com/projects/enochfiles/deploys/68f92ed921a6710008491354

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for ghost-of-hayes canceled.

Name Link
🔨 Latest commit ae0c0f8
🔍 Latest deploy log https://app.netlify.com/projects/ghost-of-hayes/deploys/68f92ed9c49440000894a068

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for acheronleaks canceled.

Name Link
🔨 Latest commit ae0c0f8
🔍 Latest deploy log https://app.netlify.com/projects/acheronleaks/deploys/68f92ed949d12d00087911bd

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for cipherstick ready!

Name Link
🔨 Latest commit ae0c0f8
🔍 Latest deploy log https://app.netlify.com/projects/cipherstick/deploys/68f92ed968c0ee000848c21b
😎 Deploy Preview https://deploy-preview-1--cipherstick.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.

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for lazarusfragment canceled.

Name Link
🔨 Latest commit ae0c0f8
🔍 Latest deploy log https://app.netlify.com/projects/lazarusfragment/deploys/68f92ed9fdb8bc00097b3cd3

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -2611,7 +2848,7 @@ const userConversions = (() => {
const endUUIDNode = ensureStringNode(endUUID ?? genUUID(), 'UUID');

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Normalize $/! grouping identifiers inside control‑flow builders

The new ConversionRuntime lets regular actions turn $foo/!bar tokens into stable UUIDs, but the special builders for Repeat/If blocks still pass grouping identifiers and UUIDs straight through to ensureStringNode/genUUID without consulting the runtime. When a JSON program now uses the advertised $name syntax for a repeat or conditional block (e.g. { action: 'Repeat', params: { GroupingIdentifier: '$loop1' } }), the generated plist embeds the literal string $loop1 for both start and end actions instead of a real UUID, while nested actions converted through buildActionsArrayFromJSON resolve $loop1 into a UUID. This mismatch breaks the block linkage and produces invalid control-flow metadata. These builders should use the shared ConversionRuntime (context) to resolve $/! tokens the same way as regular actions before emitting the XML.

Useful? React with 👍 / 👎.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant