Skip to content

feat: export ruleSides and its RuleSides type - #67

Merged
Mearman merged 1 commit into
mainfrom
feat/export-rule-sides
Nov 1, 2026
Merged

Mearman merged 1 commit into
mainfrom
feat/export-rule-sides

Conversation

@Mearman

@Mearman Mearman commented Nov 1, 2026

Copy link
Copy Markdown
Member

ruleSides already lists the paths a two-way rule reads and writes for the executor and the validator. Exporting it lets a consumer that reasons about a crosswalk's paths (whether it can be applied a page at a time, how to show its rules) cover every operation without copying which field of which rule holds a path.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ⚠️ Failed 2026-11-01T05:39:23.213643Z
We couldn’t complete this review. Comment @codex security review to try again.
34604c7 Manual 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins.

@Mearman
Mearman force-pushed the feat/export-rule-sides branch from a7ee116 to a5aac85 Compare November 1, 2026 05:33
@Mearman

Mearman commented Nov 1, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

Copy link
Copy Markdown

This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins.

ruleSides lists the paths a two-way rule reads and writes, the same list the executor and the
validator use, so a consumer that has to reason about a crosswalk's paths (to tell whether it can
be applied a page at a time, or to show its rules) handles every operation without keeping its own
copy of which field of which rule holds a path.
@Mearman
Mearman force-pushed the feat/export-rule-sides branch from a5aac85 to 34604c7 Compare November 1, 2026 05:37
@Mearman

Mearman commented Nov 1, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

Copy link
Copy Markdown

This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins.

@Mearman
Mearman merged commit a638bf0 into main Nov 1, 2026
10 checks passed
@Mearman
Mearman deleted the feat/export-rule-sides branch November 1, 2026 05:39
@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 0.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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