Skip to content

[codex] add Apple Notes public skills#150

Closed
xiaojiou176 wants to merge 5 commits intoOpenHands:mainfrom
xiaojiou176:codex/add-apple-notes-skills
Closed

[codex] add Apple Notes public skills#150
xiaojiou176 wants to merge 5 commits intoOpenHands:mainfrom
xiaojiou176:codex/add-apple-notes-skills

Conversation

@xiaojiou176
Copy link
Copy Markdown

@xiaojiou176 xiaojiou176 commented Apr 8, 2026

What these skills teach

  • notes-snapshot-control-room: how an agent acquires notesctl, separates local preflight from MCP attach, and proves the shortest control-room path without overstating hosted or marketplace claims.
  • notestorelab-case-review: how an agent installs NoteStore Lab MCP, stays on one copied case root at a time, and asks bounded evidence-backed questions instead of free-form guesses.

Why this revision is different

  • each skill now starts with what the agent learns instead of leading with disclaimers
  • install and host wiring live in references/INSTALL.md, references/OPENHANDS_MCP_CONFIG.json, and references/OPENCLAW_MCP_CONFIG.json
  • capability surface is explicit in references/CAPABILITIES.md
  • the first-success path is explicit in references/DEMO.md
  • failure handling is explicit in references/TROUBLESHOOTING.md
  • repo-local assumptions and hosted/runtime overclaims are called out as boundaries, not hidden in prose

Proof surfaces

  • the Snapshot packet now points to public proof links in the README/DEMO materials
  • the NoteStore Lab packet now points to public proof links and bounded review-safe example prompts

Validation

  • packet folders contain SKILL.md, README.md, and a full references/ set
  • local repo-side readiness checks passed before this PR refresh

Copy link
Copy Markdown
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

Thank you for the proposal!

I don’t fully understand it, sorry. What skills are you proposing to teach the agent? What is the agent supposed to know with these skills?

I think the LLM inserted too much language saying what it is not, than what it is. The contents of Skills are prompts, and I think they won’t work very well as written.

Could you show an OpenHands video using these skills?

@@ -0,0 +1,36 @@
# Usage And Proof

Use this reference when the reviewer asks what the skill helps an agent do in
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

😅 Your agent is funny, this isn’t meaningful. What was your prompt? Did you run it on the PR comments or actually prompted something?


## Best fit

- operators who want a calmer Apple Notes backup loop on their own Mac
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

“calmer”? 😅

@enyst
Copy link
Copy Markdown
Collaborator

enyst commented Apr 8, 2026

@xiaojiou176 It’s probably a good idea to just test these skills on your own custom marketplace, and then see what works. Please feel free to check the software-agent-sdk repo in this org, it has examples.

@xiaojiou176
Copy link
Copy Markdown
Author

Addressed the two main reviewer concerns in the latest push:

  1. What the skills teach
  • both packets now spell out the teaching goal in plain language via What this skill teaches, First-success path, and an explicit MCP capability surface
  • notes-snapshot-control-room now also includes a public acquisition lane for notesctl, so it no longer assumes an existing checkout
  1. How they are actually used / demo
  • both packets now include reviewer-facing usage-and-proof references
  • both packets link to live proof pages
  • notes-snapshot-control-room now embeds a GIF run-flow demo
  • notestorelab-case-review now embeds a visual demo screenshot plus the public proof path

The intent is to make these read more like reusable skill/plugin packets: install or acquire -> first success -> exact MCP surface -> truthful boundary.

@enyst enyst marked this pull request as draft April 8, 2026 22:54
@xiaojiou176
Copy link
Copy Markdown
Author

Thanks for the review.

These two skills are meant to teach an agent very concrete Apple Notes workflows, not act as generic marketing cards:

  • notes-snapshot-control-room teaches the agent how to attach the local snapshot MCP surface, start with read-only health/status tools, and answer: what should I inspect first right now?
  • notestorelab-case-review teaches the agent how to attach NoteStore Lab MCP, start from demo-safe case artifacts, and review one explicit copied case root without pretending there is a hosted Notes service.

Each folder is structured as a self-contained packet: SKILL.md, README.md, manifest.yaml, install/config snippets, capability map, demo path, and troubleshooting. The intent is: a reviewer or host can take just the skill folder and understand what the agent learns, how to attach the MCP, which tools are safe first, and what success should look like.

I also tightened the wording in the Apple Notes snapshot packet to keep it more concrete and less style-driven.

I do not have an OpenHands video attached yet; the current demo/proof paths live inside the packet references and public proof links, and I can keep tightening the packet around that reviewer expectation.

@xiaojiou176 xiaojiou176 marked this pull request as ready for review April 9, 2026 02:26
@xiaojiou176
Copy link
Copy Markdown
Author

Revision note for re-review:

  • both skills now open with What this skill teaches instead of leading with disclaimers
  • setup/config moved into explicit references/INSTALL.md, references/OPENHANDS_MCP_CONFIG.json, and references/OPENCLAW_MCP_CONFIG.json
  • capability ordering is explicit in references/CAPABILITIES.md
  • first-success flow is explicit in references/DEMO.md
  • failure checks are explicit in references/TROUBLESHOOTING.md
  • the Snapshot and NoteStore Lab packets now point at public proof links instead of only describing intent

The goal of this refresh is to make the packets read like agent capability kits, not repo promo cards. If an OpenHands-specific video is still required after this pass, I can add it in a follow-up.

@xiaojiou176
Copy link
Copy Markdown
Author

Follow-up with actual-use proof instead of only packet prose.

You asked two things on this PR:

  1. what these skills actually teach
  2. whether there is real agent use, ideally an OpenHands video

Here is the concrete answer for (1):

  • notes-snapshot-control-room teaches an agent how to prove the local Apple Notes Snapshot control room first, then attach the notesctl mcp lane, and keep AI Diagnose / Local Web API / MCP as three separate surfaces.
  • notestorelab-case-review teaches an agent how to stay on the public-safe demo path first, then review one copied case root with bounded evidence-backed questions instead of treating the live Notes store as the target.

And here is fresh actual-use proof for (2):

notes-snapshot-control-room actual use

  • Host-style prompt I used: "Use the Apple Notes Snapshot control-room skill. Start with get_status, verify_freshness, run_doctor, and list_recent_runs, then tell me whether the surface is healthy enough to keep using."
  • Fresh outputs from the repo-owned MCP surface in this workspace:
    • get_status -> health_level=OK, last_success_iso=2026-04-10T07:30:08Z, stale=no
    • verify_freshness -> OK: last success is fresh (71s)
    • run_doctor -> deterministic control-room surfaces present, launchd_loaded=true, no warnings
    • list_recent_runs -> recent_run_count=3, success_count=3, latest_status=success
    • get_access_policy -> level=OK
  • Public proof surfaces already linked from the packet are live today:

notestorelab-case-review actual use

  • Host-style prompt I used: "Start with the public-safe NoteStore Lab path. Run notes-recovery demo, notes-recovery ai-review --demo, and notes-recovery ask-case --demo --question \"What should I inspect first?\", then tell me what artifact should be inspected first."
  • Fresh outputs from the repo-owned demo path in this workspace:
    • uv run notes-recovery demo -> printed the synthetic case tree, verification preview, operator brief preview, and AI triage preview without touching private evidence
    • uv run notes-recovery ai-review --demo -> wrote a real review bundle under output/AI_Review_Demo_20260410_003121/
    • notes-recovery ask-case --demo --question "What should I inspect first?" -> answered: start with Demo operator brief at notes_recovery/resources/demo/sanitized-operator-brief.md, with cited demo artifacts and a bounded uncertainty note
    • uv run notes-recovery doctor --json -> overall_status=ready
  • Public proof surfaces already linked from the packet are live today:

Exact remaining gap

  • I still do not have a literal OpenHands screen recording attached in this thread today.
  • So if your requirement is specifically "an OpenHands UI video" rather than "actual use with exact prompts, commands, and outputs", that remaining gap is now very narrow and explicit: the missing piece is only the screen capture itself, not the workflow proof.

@enyst
Copy link
Copy Markdown
Collaborator

enyst commented Apr 11, 2026

@xiaojiou176 As explained in #154 (comment) , I think you have a simple alternative if you wish to provide skills for OpenHands users: make a custom marketplace.json and distribute it to people. Thank you for the contribution!

@enyst enyst closed this Apr 11, 2026
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