Skip to content

feat: add relayfile workspace skills#54

Merged
khaliqgant merged 3 commits into
mainfrom
codex/pr39-close-holes
May 19, 2026
Merged

feat: add relayfile workspace skills#54
khaliqgant merged 3 commits into
mainfrom
codex/pr39-close-holes

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented May 19, 2026

Summary

  • add the four PR39 relayfile workspace skills: activity-summary, daily-digest, workspace-layout, and writeback-as-files
  • register them in prpm.json and the README published-skill table without changing existing package metadata
  • add the relayfile-workspace collection for one-command installation of all four relayfile skills
  • align workspace-layout and setting-up-relayfile wording with canonical /LAYOUT.md provider manifests

This supersedes the stale wording in #39 while preserving the intended skill set.

Validation

  • node -e "JSON.parse(require('fs').readFileSync('prpm.json','utf8')); console.log('prpm ok')"
  • package metadata comparison against origin/main: existing package entries unchanged, only activity-summary/daily-digest/writeback-as-files/workspace-layout added
  • collection validation: every @agent-relay package reference resolves and relayfile-workspace contains the four required relayfile skills
  • rg -n ".layout.md|/.layout|/.layout" README.md prpm.json skills || true

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

The PR publishes four new skills to guide both agents and adapter authors through relayfile mount operations: digest-based activity querying, mount navigation, and file-based mutations. The skills are registered in README and prpm.json, with a minor layout documentation correction to setting-up-relayfile.

Changes

New skills: digests, layout navigation, and mutations

Layer / File(s) Summary
Skills manifest and catalog registration
README.md, prpm.json, skills/setting-up-relayfile/SKILL.md
README "Published Skills" table and prpm.json packages array register the four new skills (activity-summary, daily-digest, workspace-layout, writeback-as-files) with versions, descriptions, tags, and SKILL.md paths. The setting-up-relayfile documentation is updated to reference <provider>/LAYOUT.md instead of <provider>/.layout.md.
Activity summary skill: digest-first agent guidance
skills/activity-summary/SKILL.md
Instructs agents to read pre-computed deterministic activity digests from <mount>/digests/ for time-windowed activity questions. Documents digest file structure, coverage verification workflow, fallback scenarios for uncovered windows or custom filters, and the observed reduction in tool calls when digests are preferred.
Daily digest skill: authoring contract for adapters
skills/daily-digest/SKILL.md
Specifies the digest authoring contract: directory structure, window types and mutability rules, required header fields, per-provider Markdown format (alphabetical sections, time-sorted bullets, canonical identifiers, mount links), the async digest(ctx: DigestContext) adapter function signature, null vs exception semantics, regeneration rules for rolling and closing windows, and common authoring mistakes.
Workspace layout skill: mount navigation guidance
skills/workspace-layout/SKILL.md
Describes how to navigate relayfile mounts: read root and provider-specific LAYOUT.md files, use by-* alias indexes for resource lookup, understand the <identifier>__<uuid>.<ext> filename convention, and apply concrete lookup patterns by title, date, state, or UUID. Includes explicit "do not" guidance against recursive grep, hardcoded paths, and ignoring provider layouts.
Writeback-as-files skill: mutation workflow guide
skills/writeback-as-files/SKILL.md
Explains the writeback-by-file workflow end-to-end: drop JSON payloads into canonical writeback directories, discover correct paths and schemas via .schema.json siblings, use filename conventions for idempotency and retries, monitor progress with relayfile writeback list and relayfile status, recover from dead-lettered writes via .error.json, and follow constraints (no SDK calls, no read-only writes, no tight-polling).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • AgentWorkforce/skills#36: Both PRs modify setting-up-relayfile skill documentation and its prpm.json metadata around the layout-file convention.
  • AgentWorkforce/skills#30: Both PRs update skills/setting-up-relayfile/SKILL.md documentation for relayfile mount discovery and the LAYOUT.md filename convention.
  • AgentWorkforce/skills#38: Both PRs modify the README "Published Skills" table—this PR extends it with four new skill entries, while the related PR structures the table itself.

Poem

🐰 Four skills hop into view,
Digests guide agents through,
Mounts laid plain, writes in files,
Relayfile grins its widest smiles! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: add relayfile workspace skills' directly and clearly summarizes the main change—adding four new workspace skills to the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The pull request description is clearly related to the changeset, describing the addition of four relayfile workspace skills, their registration in prpm.json and README, and documentation updates.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/pr39-close-holes

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

View 5 additional findings in Devin Review.

Open in Devin Review

Comment thread prpm.json Outdated
],
"files": [
"skills/orchestrating-agent-relay/SKILL.md"
"skills/running-headless-orchestrator/SKILL.md"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔴 prpm.json references non-existent file skills/running-headless-orchestrator/SKILL.md

The skill was renamed from orchestrating-agent-relay to running-headless-orchestrator in prpm.json (line 91–105), but the actual directory on disk is still skills/orchestrating-agent-relay/ — no skills/running-headless-orchestrator/ directory exists anywhere in the repo. This means the files entry at line 105 points to a path that doesn't exist, which will break any tooling (e.g. prpm install) that resolves packages by reading the files array in the manifest.

Prompt for agents
The prpm.json files entry at line 105 references skills/running-headless-orchestrator/SKILL.md but the directory on disk is still skills/orchestrating-agent-relay/SKILL.md. Either rename the directory from skills/orchestrating-agent-relay to skills/running-headless-orchestrator (and update the SKILL.md frontmatter name field inside it), or revert the files path in prpm.json back to skills/orchestrating-agent-relay/SKILL.md. The README.md at line 15 has the same broken link and needs to be updated to match whichever path is chosen.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment thread README.md Outdated
| [relay-80-100-workflow](skills/relay-80-100-workflow/SKILL.md) | 1.0.7 | Author workflows that close the 80-to-100 validation gap with repair-aware test, verify, mandatory Claude-then-Codex review/fix with test hardening, and commit gates. |
| [review-fix-signoff-loop](skills/review-fix-signoff-loop/SKILL.md) | 1.0.2 | Loop review, repair, validation, and fresh-context dual-agent signoff until independent reviewers both satisfy the verdict contract. |
| [using-agent-relay](skills/using-agent-relay/SKILL.md) | 1.2.0 | Coordinate agents in real time with Relaycast messaging, channels, threads, reactions, search, and webhooks. |
| [running-headless-orchestrator](skills/running-headless-orchestrator/SKILL.md) | 1.0.4 | Self-bootstrap Agent Relay infrastructure and manage worker agents without human intervention. |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔴 README links to non-existent skills/running-headless-orchestrator/SKILL.md

The README skill table at line 15 links to skills/running-headless-orchestrator/SKILL.md, but this path does not exist. The actual file is at skills/orchestrating-agent-relay/SKILL.md. This is the same rename inconsistency as in prpm.json:105 — the metadata was updated but the directory was never renamed.

Suggested change
| [running-headless-orchestrator](skills/running-headless-orchestrator/SKILL.md) | 1.0.4 | Self-bootstrap Agent Relay infrastructure and manage worker agents without human intervention. |
| [running-headless-orchestrator](skills/orchestrating-agent-relay/SKILL.md) | 1.0.4 | Self-bootstrap Agent Relay infrastructure and manage worker agents without human intervention. |
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@khaliqgant khaliqgant merged commit d514827 into main May 19, 2026
2 checks passed
@khaliqgant khaliqgant deleted the codex/pr39-close-holes branch May 19, 2026 18:25
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