diff --git a/crates/quarry-server/resources/prompt.md b/crates/quarry-server/resources/prompt.md index b7bef46..8262ed4 100644 --- a/crates/quarry-server/resources/prompt.md +++ b/crates/quarry-server/resources/prompt.md @@ -17,7 +17,7 @@ quarry open "/absolute/path/to/file.md" ### Editing existing documents -When reviewing substantive changes to an existing Markdown document, first open it in Quarry and use suggestions for proposals or unsolicited changes. Apply a direct edit only when the user asks for that concrete change. +When reviewing substantive changes to an existing Markdown document, first open it in Quarry and use suggestions for proposals or unsolicited changes. Apply a direct edit only when the user asks for that concrete change. Leave suggestions for the user to accept or reject. When the review process is complete, sync the content from Quarry back to the file on disk. diff --git a/crates/quarry-server/resources/quarry.SKILL.md b/crates/quarry-server/resources/quarry.SKILL.md index cec93a6..b818674 100644 --- a/crates/quarry-server/resources/quarry.SKILL.md +++ b/crates/quarry-server/resources/quarry.SKILL.md @@ -45,8 +45,9 @@ A concrete imperative Quarry comment such as “Add this section,” “Change t wording,” or “Remove this block” is a direct-edit instruction for that scoped change; do the work, reply to the comment, and resolve the addressed thread. Do not respond only with a promise or proposal. Unsolicited changes, and changes -the user explicitly asks to review before applying, stay as suggestions. Both -operation families share the same transaction envelope. To author or +the user explicitly asks to review before applying, stay as suggestions. +Leave suggestions for the user to accept or reject. +Both operation families share the same transaction envelope. To author or restructure a whole document, prefer the Markdown `PUT` (see Whole-Document Markdown Writes) over hand-assembling block ops. diff --git a/crates/quarry-server/tests/rest_agent_discovery.rs b/crates/quarry-server/tests/rest_agent_discovery.rs index a02d9a2..0840257 100644 --- a/crates/quarry-server/tests/rest_agent_discovery.rs +++ b/crates/quarry-server/tests/rest_agent_discovery.rs @@ -159,6 +159,7 @@ async fn agent_discovery_endpoints_expose_skill_docs_and_metadata() -> anyhow::R assert!(surface.contains("it does not authorize direct content edits")); assert!(surface.contains("If no concrete Quarry task exists")); } + assert!(skill.contains("Leave suggestions for the user to accept or reject")); assert!(!skill.contains("Wait for the user's instruction")); assert!(!docs.contains("instruction in the same request")); assert!(docs.contains("/v1/tmp/documents/$SECRET")); diff --git a/crates/quarry-server/tests/rest_onboarding.rs b/crates/quarry-server/tests/rest_onboarding.rs index cfdf132..f00bc44 100644 --- a/crates/quarry-server/tests/rest_onboarding.rs +++ b/crates/quarry-server/tests/rest_onboarding.rs @@ -95,6 +95,7 @@ async fn prompt_document_teaches_the_review_workflow() { assert!(body.contains("quarry open")); assert!(body.contains("### Share a newly written document")); assert!(body.contains("### Editing existing documents")); + assert!(body.contains("Leave suggestions for the user to accept or reject")); assert!(body.contains("sync the content from Quarry back to the file on disk")); assert!(body.contains("creates the shared document")); assert!(body.contains("Instructions given before the document opens remain valid"));