docs(skills): add four workspace-primitive skills#146
Closed
khaliqgant wants to merge 1 commit into
Closed
Conversation
Bundles activity-summary, daily-digest, writeback-as-files, and workspace-layout into .claude/skills/ so agents working in this repo pick them up automatically. - activity-summary: read digests/yesterday.md before crawling provider data for time-windowed activity questions - daily-digest: authoring contract for <mount>/digests/ including adapter digest() exports and regeneration rules - writeback-as-files: drop-a-JSON-file writeback contract with schema discovery and dead-letter recovery - workspace-layout: LAYOUT.md + by-* alias indexes instead of find/grep -r Each is also being published from AgentWorkforce/skills and shipped into the cloud sandbox via parallel PRs. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughThis PR adds four new agent skill documentation files to ChangesAgent Skill Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/activity-summary/SKILL.md:
- Line 30: Update the mutability label and description for the `yesterday.md`
entry in SKILL.md so it matches the digest contract: change the "rolling" tag to
indicate immutability (e.g., "immutable" or "generated at day close") and
clarify the text to state that `yesterday.md` is generated once at the prior
calendar day's close and does not change thereafter; update the line containing
`yesterday.md # rolling — generated at 00:00 local for the prior calendar
day` to reflect the immutable/once-generated semantics so agents read the
correct contract.
- Around line 55-57: Under the "## slack" section update the lone bullet "- 7
messages in `#gtm-prospects` mentioning "ACME"" to follow the file's "canonical
link per bullet" pattern: replace the plain channel name with a markdown link
that points to the canonical mount URL for the Slack channel (i.e., use the same
canonical-link format used by other providers in this file). Edit the bullet
under the "## slack" header so it includes the canonical mount link for
`#gtm-prospects` while preserving the rest of the text.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 17ded618-e9bc-4165-bda7-b754ceccf73e
📒 Files selected for processing (4)
.claude/skills/activity-summary/SKILL.md.claude/skills/daily-digest/SKILL.md.claude/skills/workspace-layout/SKILL.md.claude/skills/writeback-as-files/SKILL.md
|
|
||
| ```bash | ||
| $ ls $MOUNT/digests/ | ||
| yesterday.md # rolling — generated at 00:00 local for the prior calendar day |
There was a problem hiding this comment.
Fix yesterday.md mutability label to match the digest contract.
yesterday.md is described as “rolling” here, but the contract defines it as generated at day close and then immutable. This mismatch can lead to wrong agent behavior.
Suggested doc fix
-yesterday.md # rolling — generated at 00:00 local for the prior calendar day
+yesterday.md # closing — generated at 00:00 local for the prior calendar day; immutable afterward📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| yesterday.md # rolling — generated at 00:00 local for the prior calendar day | |
| yesterday.md # closing — generated at 00:00 local for the prior calendar day; immutable afterward |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.claude/skills/activity-summary/SKILL.md at line 30, Update the mutability
label and description for the `yesterday.md` entry in SKILL.md so it matches the
digest contract: change the "rolling" tag to indicate immutability (e.g.,
"immutable" or "generated at day close") and clarify the text to state that
`yesterday.md` is generated once at the prior calendar day's close and does not
change thereafter; update the line containing `yesterday.md # rolling —
generated at 00:00 local for the prior calendar day` to reflect the
immutable/once-generated semantics so agents read the correct contract.
Comment on lines
+55
to
+57
| ## slack | ||
| - 7 messages in #gtm-prospects mentioning "ACME" | ||
| ``` |
There was a problem hiding this comment.
Keep provider examples consistent with the “canonical link per bullet” rule.
The Slack bullet is the only one without a canonical mount link, which contradicts the format guidance in the same file.
Suggested doc fix
## slack
-- 7 messages in `#gtm-prospects` mentioning "ACME"
+- 7 messages in `#gtm-prospects` mentioning "ACME" — [/slack/channels/C0ADE9B71CN__gtm-prospects/messages/2026-05-12.jsonl]📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| ## slack | |
| - 7 messages in #gtm-prospects mentioning "ACME" | |
| ``` | |
| ## slack | |
| - 7 messages in `#gtm-prospects` mentioning "ACME" — [/slack/channels/C0ADE9B71CN__gtm-prospects/messages/2026-05-12.jsonl] |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.claude/skills/activity-summary/SKILL.md around lines 55 - 57, Under the "##
slack" section update the lone bullet "- 7 messages in `#gtm-prospects` mentioning
"ACME"" to follow the file's "canonical link per bullet" pattern: replace the
plain channel name with a markdown link that points to the canonical mount URL
for the Slack channel (i.e., use the same canonical-link format used by other
providers in this file). Edit the bullet under the "## slack" header so it
includes the canonical mount link for `#gtm-prospects` while preserving the rest
of the text.
4 tasks
Member
Author
|
Closing in favor of AgentWorkforce/skills#39 (skills live in the central registry, not bundled into relayfile/.claude/skills) and #147 (specs for the missing primitives the skills assume). Re-opening once the skills become accurate against shipping code is unnecessary — they're maintained from skills#39. |
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
1 participant
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.
Summary
Adds four workspace-primitive skills under
.claude/skills/so agents working in this repo pick them up automatically. These are the conceptual primitives referenced in the Just Give the Agent Files blog post.digests/yesterday.mdbefore crawling provider data for time-windowed activity questions. Cuts the canonical "what did I work on yesterday" task from ~25 tool calls to ~4 in our benchmarks.<mount>/digests/: window taxonomy, per-provider section format, adapterdigest()exports, regeneration rules, determinism requirements..schema.json, provider mutation lands in ~30s. Coversrelayfile writeback listand.relay/dead-letter/recovery.LAYOUT.md, then<provider>/.layout.md, thenby-*alias subtrees. Documents the<identifier>__<uuid>filename convention.Companion PRs:
AgentWorkforce/skills#39— publishes the four skills as@agent-relay/*packagesAgentWorkforce/cloud— ships them in the sandboxTest plan
.claude/skills/<slug>/SKILL.mdfiles have valid frontmatternamematches the directory name<identifier>__<uuid>anddigests/references stay consistent withdocs/LAYOUT.md🤖 Generated with Claude Code