Skip to content

TACo condition authoring skill#774

Open
cygnusv wants to merge 1 commit intonucypher:signing-epicfrom
cygnusv:skill
Open

TACo condition authoring skill#774
cygnusv wants to merge 1 commit intonucypher:signing-epicfrom
cygnusv:skill

Conversation

@cygnusv
Copy link
Copy Markdown
Member

@cygnusv cygnusv commented Apr 13, 2026

Type of PR:

  • Bugfix
  • Feature
  • Documentation
  • Other

Required reviews:

  • 1
  • 2
  • 3

What this does:

High-level idea of the changes introduced in this PR. List relevant API
changes (if any), as well as related PRs and issues.

Issues fixed/closed:

  • Fixes #...

Why it's needed:

Explain how this PR fits in the greater context of the NuCypher Network. E.g.,
if this PR address a nucypher/productdev issue, let reviewers know!

Notes for reviewers:

Depends on nucypher/taco-docs#12, so don't merge it before.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 13, 2026

Deploy Preview for taco-demo ready!

Name Link
🔨 Latest commit 1ad9ff4
🔍 Latest deploy log https://app.netlify.com/projects/taco-demo/deploys/69dcc56edc7559000879f46d
😎 Deploy Preview https://deploy-preview-774--taco-demo.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
Copy link
Copy Markdown

netlify bot commented Apr 13, 2026

Deploy Preview for taco-nft-demo ready!

Name Link
🔨 Latest commit 1ad9ff4
🔍 Latest deploy log https://app.netlify.com/projects/taco-nft-demo/deploys/69dcc56e34908c0008485770
😎 Deploy Preview https://deploy-preview-774--taco-nft-demo.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.

@cygnusv cygnusv changed the base branch from main to signing-epic April 13, 2026 10:29
Comment on lines +41 to +44
https://raw.githubusercontent.com/nucypher/taco-web/signing-epic/packages/taco/schema-docs/condition-schema.json

2. Human-readable schema reference:
https://github.com/nucypher/taco-web/blob/signing-epic/packages/taco/schema-docs/condition-schemas.md
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should think about what branch is best to reference in order to be the most evergreen. This currently uses signing-epic but eventually that branch will be merged into epic-v0.7.x and then into main - #657.

We can probably leave as is, and have a follow-up PR to update the reference when applicable.

Just pointing this out to remind us.

- After the `:`, the name must match /^[a-zA-Z_][a-zA-Z0-9_]*$/ (letters, digits, underscores only).
- No dots, dashes, hyphens, or spaces. `:user-address`, `:user.address`, `:1first` are all INVALID.
- Case-sensitive: `:userAddress` ≠ `:useraddress`.
- Built-in variables you do NOT need to define yourself: `:userAddress` (requester's wallet), `:message` (signed message), `:signature` (signature being verified), `:jwtToken` (JWT being validated), `:signingConditionObject` (UserOperation in threshold signing).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We probably need to be a little clearer here - see nucypher/taco-docs#12 (comment).

## Q&A

**Q: Can I test conditions before deploying?**
A: Yes. Use the [TACo Playground](https://playground.taco.build/) to simulate decryption with your condition and test requesters' data.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We'll need to bring the TACo Playground up to date.

A: Yes. Any context variable (starting with `:`) is valid. You supply its value at decryption time. Tell the LLM: "Use a context variable `:myCustomVar` and I will supply its value at runtime."

**Q: Are there limits on condition complexity?**
A: CompoundCondition max 5 operands, max 2 nesting depth. SequentialCondition handles 2–20 steps. If you need more: compound → operand1 (SequentialCondition), operand2 (SequentialCondition), etc.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@derekpierre
Copy link
Copy Markdown
Member

derekpierre commented Apr 14, 2026

Very cool - nice work.

You'll also need to rebase over signing-epic, then the CI failures should go away.

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.

2 participants