Skip to content

Improve draft-planning skill with collaborative workflow#12

Merged
leeovery merged 2 commits intomainfrom
claude/improve-draft-plan-skill-01QY6aVxjf5nNVwNUAh1cUoj
Dec 11, 2025
Merged

Improve draft-planning skill with collaborative workflow#12
leeovery merged 2 commits intomainfrom
claude/improve-draft-plan-skill-01QY6aVxjf5nNVwNUAh1cUoj

Conversation

@leeovery
Copy link
Copy Markdown
Owner

Replaces form-filling approach with present-then-approve workflow:

  • Claude synthesizes ALL reference material, presents for validation
  • Human reviews, refines, adds missing info through discussion
  • Only logs to draft when explicitly approved
  • Treats all source material as untrusted input

Replaces form-filling approach with present-then-approve workflow:
- Claude synthesizes ALL reference material, presents for validation
- Human reviews, refines, adds missing info through discussion
- Only logs to draft when explicitly approved
- Treats all source material as untrusted input
- Draft plan must be standalone - formal planning uses only this document
- Added course correction to refine step - adjusting based on subsequent project work
- User brings context that may not be in reference material
@leeovery leeovery merged commit ca8996e into main Dec 11, 2025
@leeovery leeovery deleted the claude/improve-draft-plan-skill-01QY6aVxjf5nNVwNUAh1cUoj branch December 11, 2025 21:06
leeovery added a commit that referenced this pull request Apr 20, 2026
…fig null unset

Three small correctness/style fixes bundled:

- OpenAIProvider.embed() now throws a descriptive error if OpenAI
  returns an empty data array instead of a cryptic TypeError reading
  res.data[0].embedding. Unlikely in practice but costs nothing to
  guard. (deferred #13)

- OpenAIProvider.embedBatch() no longer mutates res.data in place.
  Spread before sort. (deferred #12)

- config.loadConfig() now treats explicit null in system/project config
  as an unset sentinel, letting a project config clear a system-level
  default. (deferred #14)

Closes deferred-issues #12, #13, #14.
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