feat(routines): B-0507 and B-0508 cloud-schedule schema definition#3937
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
Lands slices 1 and 2 of B-0448 (Cloud Routines): a research doc capturing the Anthropic Cloud Routines API/auth/trigger surface, plus a schema and TypeScript types laying the groundwork for cloud-schedule.json files alongside the existing Desktop schedule.json.
Changes:
- Adds
tools/routines/cloud-schedule.schema.json(taskId + trigger oneOf: scheduled/github_event/api, plus repos/connectors/notes). - Exports
CloudTriggerandCloudScheduleResulttypes fromtools/routines/install.ts(no reader/consumer wired yet). - Adds research doc and closes B-0507/B-0508 backlog rows; updates
tools/routines/README.mdto documentcloud-schedule.json.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/routines/cloud-schedule.schema.json | New JSON Schema describing cloud routine config. |
| tools/routines/install.ts | Adds exported CloudTrigger/CloudScheduleResult types (not yet consumed). |
| tools/routines/README.md | Documents new cloud-schedule.json file. |
| docs/research/2026-05-14-cloud-routines-api-auth-registration-surface-b0507.md | B-0507 research write-up. |
| docs/backlog/P1/B-0507-...md | Marks B-0507 closed; acceptance criteria checked. |
| docs/backlog/P1/B-0508-...md | Marks B-0508 closed; acceptance criteria checked. |
…#3937) Address two of three Copilot findings on PR #3937: 1. Relax `taskId` from required to optional. install.ts derives the canonical taskId from the containing directory name (syncRoutine in install.ts:131); the schema's `required` constraint contradicted that derivation contract. Expand the field description to name the directory as authoritative and explain why the field is preserved (self-documentation + cloud-side tooling without filesystem layout). 2. Disambiguate the two `repos` arrays via expanded descriptions rather than renaming. Trigger-scope `trigger.repos` selects webhook source repos (github_event only); runtime-scope top-level `repos` controls routine environment repo access. Same shape, distinct semantics; rename would have churned a freshly-authored schema for clarity that descriptions can supply. Third finding (date-prefix research filename matching markdownlint ignore pattern) is resolved per the existing B-0078 policy (.markdownlint-cli2.jsonc:85-114) which intentionally keeps the broad `docs/research/2026-*-*.md` pattern; the theoretical concern about authored content silently skipping lint was empirically outweighed by 82+ legitimate verbatim-ferry violations that the narrow pattern would have surfaced. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Merged
4 tasks
AceHack
added a commit
that referenced
this pull request
May 16, 2026
CI failures on PR #3941: - markdownlint MD029 line 22 (blockquote ordered list starting at "2." preserves source numbering from B-0543's 4-step proof strategy; added <!-- markdownlint-disable-next-line MD029 --> + clarifying intro) - markdownlint MD047 line 102 (missing trailing newline) - BACKLOG.md generated-index drift (B-0562 row added; incidentally sweeps up pre-existing B-0507/B-0508 [ ] → [x] flip that should have landed with PR #3937 but didn't trigger a regen) Co-Authored-By: Claude <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
May 16, 2026
…kson → HaPPY-like QECC (#3941) * research(B-0562): QG isomorphism Step 2 — cube + Adinkra + Cayley-Dickson → HaPPY-like QECC Step 2 of the 4-step QG isomorphism proof path opened in B-0543 / PR #3614. Step 1 (B-0544, PR #3614) formalized Zeta_{RA} = (Zeta, M, A) with an internal monad M for memory and modal operator A for attention. Step 2 unfolds the two axioms into a 4-axis cube (Remember × When × Pay × Attention), grafts the Adinkra-Gates supermultiplet layer onto cube vertices, lifts the cube through a Cayley-Dickson tower (R → C → H → O → S → T), and proposes the algebraic shape that matches HaPPY (holographic perfect-tensor) QEC codes. Originally allocated as B-0545; collided with PR #3619's renumber-sweep that re-took B-0545 for Riven's cursor-terminal loop work. Renumbered to B-0562 (next free above all merged-on-main + in-flight #3878's B-0561) per the multi-Otto ID-allocation discipline in .claude/rules/otto-channels-reference-card.md. Crash-recovery note: this row + research file were the only artifacts from the pre-crash Otto session that hadn't already shipped via concurrent PRs (rule landed via #3935, B-0507 follow-on via #3937, Lior tick via #3936). Per-artifact refresh-before-decide caught the duplications before pushing. Co-Authored-By: Claude <noreply@anthropic.com> * fix(b-0562): MD029/MD047 lint + BACKLOG.md index regen CI failures on PR #3941: - markdownlint MD029 line 22 (blockquote ordered list starting at "2." preserves source numbering from B-0543's 4-step proof strategy; added <!-- markdownlint-disable-next-line MD029 --> + clarifying intro) - markdownlint MD047 line 102 (missing trailing newline) - BACKLOG.md generated-index drift (B-0562 row added; incidentally sweeps up pre-existing B-0507/B-0508 [ ] → [x] flip that should have landed with PR #3937 but didn't trigger a regen) Co-Authored-By: Claude <noreply@anthropic.com> * fix(b-0562): markdownlint MD029 + MD047 — bold-prose + trailing newline Resolves Copilot review threads on PR #3941: - L22: ordered-list `> 2.` in blockquote → bold `> **Step 2.**` (MD029) - L102: missing trailing newline (MD047) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
This was referenced May 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Delivers slice 1 and 2 of B-0448 (Cloud Routines).