Skip to content

onboarding: point the premise-probing doctrine at the probe skill - #62

Merged
alongd merged 1 commit into
mainfrom
probe-pointer
Aug 11, 2026
Merged

onboarding: point the premise-probing doctrine at the probe skill#62
alongd merged 1 commit into
mainfrom
probe-pointer

Conversation

@alongd

@alongd alongd commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

CLAUDE.global.md's "Probe the premise before building" section states the doctrine but not the procedure, so each session re-derives how to actually run one.

Adds two lines pointing at the new probe skill (DanaResearchGroup/agent-skills#62), which carries the procedure: picking the load-bearing premise, trying to refute it by one line of arithmetic before commissioning a search, re-running prior art from scratch when the framing moves fields, and reporting what the probe could NOT reach alongside what it found.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the onboarding doctrine on premise-probing to point readers to a concrete, reusable procedure (the probe skill), reducing repeated re-derivation of how to run a probe in each session.

Changes:

  • Adds a brief sentence in the “Probe the premise before building” section that directs users to the probe skill as the procedural reference.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

The section states the doctrine but not the procedure. The probe skill in
agent-skills carries it: picking the load-bearing premise, refuting by
arithmetic before commissioning a search, re-running prior art when the
framing moves fields, and reporting what the probe could not reach.
@alongd

alongd commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Codex reviewed this alongside the two skills PRs it depends on, and raised the one thing that makes a global instruction file different from a skill: it is loaded by every session of every group member, including those who have not installed the group's agent-skills clone. As written it pointed at a procedure that, for them, is simply unavailable — which is worse than the re-derivation the pointer was meant to prevent.

Added a fallback: if the skill isn't installed (ONBOARDING step 5), the sentence itself is the checklist and gets run inline. The core doctrine above it was already self-contained, so nobody is left worse off than before.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

onboarding/CLAUDE.global.md:134

  • This text assumes the probe skill is available once skills are installed, but the referenced upstream change (agent-skills#62) may not have landed in the reader's local agent-skills checkout yet. As written, someone who has installed skills could still not find probe and be confused about what to do.
The procedure this doctrine implies lives in the `probe` skill: which premise to pick, trying to refute it by one line of arithmetic before commissioning a search, re-running prior art from scratch when the framing moves fields, and reporting what the probe could NOT reach alongside what it found. If the skill isn't installed (ONBOARDING step 5), that sentence is the checklist — run it inline rather than treating the procedure as unavailable.

@alongd
alongd merged commit 3bbb188 into main Aug 11, 2026
5 checks passed
@alongd
alongd deleted the probe-pointer branch August 11, 2026 09:35
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