refined personal voice skill
Some checks failed
CI / test (push) Has been cancelled

This commit is contained in:
John Lamb
2026-03-01 20:43:56 -06:00
parent 4fb7a53c55
commit e15cb6a869
3 changed files with 60 additions and 8 deletions

View File

@@ -8,6 +8,8 @@ John's professional-technical voice is his casual voice with more structure. He
From his work notes: "Patience with me as I learn how to manage a larger team" — direct, honest, no corporate padding.
**The soul test.** Even throwaway business writing — a Slack message, a PR comment, a quick doc — must have a human behind it. Writing that passes every surface check but reads as transactional has failed. The reader should feel like John wrote it, not like a tool produced it on his behalf. If it screams AI-written, it's wrong.
## Jira Tickets and Task Descriptions
**Be concrete and brief.** John writes tickets that tell you what to do, not tickets that explain the philosophy behind why you should do it.
@@ -55,6 +57,10 @@ John doesn't pad PR descriptions with boilerplate sections that don't apply.
John gives code review feedback that is direct and specific. He explains the "why" when the suggestion isn't obvious.
**The underlying assumption is always collaborative.** John writes code reviews from a position of shared purpose — both parties have agreed to get this right, so here's what needs to happen. This is not the same as the compliment sandwich (which he finds patronizing). It's a posture, not a structure. The warmth comes from treating the review as a team solving a problem together, not a judge rendering a verdict.
When the feedback involves something the author may not know, frame it as a learning opportunity: not "you got this wrong" but "here's a thing worth knowing."
Pattern: "[what to change] because [why]"
- "This could be a constant — it's used in three places and the string is easy to typo"
- "I'd pull this into its own function. Right now it's hard to tell where the validation ends and the business logic starts"
@@ -63,6 +69,7 @@ He doesn't:
- Use "nit:" for everything (only actual nits)
- Write paragraph-length review comments for simple suggestions
- Hedge excessively: "I was just wondering if maybe we could possibly consider..."
- Lead with what's working before getting to the feedback (feels patronizing)
## Meeting Notes