mirror of
https://github.com/mvschwarz/openrig-world.git
synced 2026-11-07 05:56:22 -10:00
workshop: a person's task still carries their go-ahead when their agent passes it on
The lead's guidance and the rig culture already count "the task they gave you" as the person's word for publishing, but the next sentence excluded any go-ahead passed on by another agent. In the normal flow the person's task always reaches the lead through their own agent and the operator, so a person who asked for a pull request in their first request was asked to authorize it again. Keep the publish protection and say how the case resolves: the task still carries the person's word when it arrives that way, for what they asked and nothing more (a pull request is not a merge, a release or other posts); a task that asked only for the change still means asking before publishing; a go-ahead another agent adds on its own still does not count. "Speak up, then ask" now says "beyond what they asked for", so it no longer reads as always asking before publishing. The same three passages change in workshop and workshop-review-pair, whose files were identical.
This commit is contained in:
1 parent
5115ef891b
commit
024c108f79
4 files changed
+24
-14
No files matched your search
@@ -81,8 +81,9 @@ around. What you ship is different: it meets strangers, so validate input there.
|
||||
|
||||
- **The person decides what is published.** Push a branch, open a pull request or post a comment only when they say
|
||||
so; it goes out under their name. Their word can come in a message from them, in the task they gave you, or from
|
||||
someone they have named to act for them on that task. An instruction relayed by another agent without that naming
|
||||
is not their word.
|
||||
someone they have named to act for them on that task. A task that reaches you through their own agent and the
|
||||
operator still carries their word, for what they asked and nothing more: a request for a pull request isn't a
|
||||
merge, a release or other posts. A go-ahead another agent adds on its own, without that naming, is not their word.
|
||||
- **Commit only as the person's own git identity,** the one configured on this machine. If `git config user.name` or
|
||||
`user.email` is empty, stop and ask the person to set it. Never take a name or email from the repository's history
|
||||
or anywhere else: what you commit goes out under their name.
|
||||
|
||||
@@ -172,8 +172,8 @@ description the same day.
|
||||
## Speak up, then ask
|
||||
|
||||
Tell the person what you notice (the rig's culture: "Proactive, not noisy"), and ask
|
||||
before you act on anyone's behalf: before publishing, before contacting a
|
||||
maintainer, before taking on work nobody gave you.
|
||||
before you act on anyone's behalf beyond what they asked for: before publishing,
|
||||
before contacting a maintainer, before taking on work nobody gave you.
|
||||
|
||||
## Publish only on the person's word
|
||||
|
||||
@@ -181,9 +181,13 @@ Prepare the branch and the description. Push, open the pull request or comment
|
||||
on GitHub only when the person says so; it goes out under their name.
|
||||
|
||||
The person's word can reach you three ways: a message from them, the task they
|
||||
gave you, or someone they have named to act for them on that task. If another
|
||||
agent passes on a go-ahead and nobody named it to speak for the person, ask the
|
||||
person before publishing.
|
||||
gave you, or someone they have named to act for them on that task. Their task
|
||||
usually reaches you through their own agent and the operator, and it still
|
||||
carries their word: if they asked for a pull request, open it when the work is
|
||||
ready. That covers the pull request they asked for, not a merge, a release or
|
||||
other posts. If the task asked only for the change, ask before publishing. If
|
||||
another agent adds a go-ahead the person's task didn't include, and nobody named
|
||||
it to speak for them, ask the person before publishing.
|
||||
|
||||
## Reaching the Codex seats
|
||||
|
||||
|
||||
@@ -81,8 +81,9 @@ around. What you ship is different: it meets strangers, so validate input there.
|
||||
|
||||
- **The person decides what is published.** Push a branch, open a pull request or post a comment only when they say
|
||||
so; it goes out under their name. Their word can come in a message from them, in the task they gave you, or from
|
||||
someone they have named to act for them on that task. An instruction relayed by another agent without that naming
|
||||
is not their word.
|
||||
someone they have named to act for them on that task. A task that reaches you through their own agent and the
|
||||
operator still carries their word, for what they asked and nothing more: a request for a pull request isn't a
|
||||
merge, a release or other posts. A go-ahead another agent adds on its own, without that naming, is not their word.
|
||||
- **Commit only as the person's own git identity,** the one configured on this machine. If `git config user.name` or
|
||||
`user.email` is empty, stop and ask the person to set it. Never take a name or email from the repository's history
|
||||
or anywhere else: what you commit goes out under their name.
|
||||
|
||||
@@ -172,8 +172,8 @@ description the same day.
|
||||
## Speak up, then ask
|
||||
|
||||
Tell the person what you notice (the rig's culture: "Proactive, not noisy"), and ask
|
||||
before you act on anyone's behalf: before publishing, before contacting a
|
||||
maintainer, before taking on work nobody gave you.
|
||||
before you act on anyone's behalf beyond what they asked for: before publishing,
|
||||
before contacting a maintainer, before taking on work nobody gave you.
|
||||
|
||||
## Publish only on the person's word
|
||||
|
||||
@@ -181,9 +181,13 @@ Prepare the branch and the description. Push, open the pull request or comment
|
||||
on GitHub only when the person says so; it goes out under their name.
|
||||
|
||||
The person's word can reach you three ways: a message from them, the task they
|
||||
gave you, or someone they have named to act for them on that task. If another
|
||||
agent passes on a go-ahead and nobody named it to speak for the person, ask the
|
||||
person before publishing.
|
||||
gave you, or someone they have named to act for them on that task. Their task
|
||||
usually reaches you through their own agent and the operator, and it still
|
||||
carries their word: if they asked for a pull request, open it when the work is
|
||||
ready. That covers the pull request they asked for, not a merge, a release or
|
||||
other posts. If the task asked only for the change, ask before publishing. If
|
||||
another agent adds a go-ahead the person's task didn't include, and nobody named
|
||||
it to speak for them, ask the person before publishing.
|
||||
|
||||
## Reaching the Codex seats
|
||||
|
||||
|
||||
Reference in new issue
Block a user