Add philosophy, dogfood guide, and evidence-log template (re-target to main) #33

Merged
pdurlej merged 4 commits from docs/dogfood-and-philosophy into main 2026-05-05 11:32:00 +02:00
Collaborator

Re-targeting branch docs/dogfood-and-philosophy to main.

Why this PR exists

Original PR #3 with the same content was merged on 2026-05-04 — but its base was feat/phase-a-ship-blockers, not main. After Phase A merge (PR #2), feat/phase-a-ship-blockers was already on main, so subsequent merge into it didn't propagate the docs to main. The three documents (docs/philosophy.md, docs/dogfood.md, docs/dogfood-log-template.md) have been accessible only via the feat branch, not on main.

This was discovered today during PR #32 review by Codex (mandatory non-author reviewer per ADR 0010 — first live test of the rule, working as designed). PR #32's AGENTS.md references these three documents; the references resolve 404 on main because the docs aren't there yet. Codex blocked PR #32 with request_changes. Resolving that blocker requires getting the docs on main first.

This PR carries the same single commit as PR #3 (4e06c4a3), now targeted at main directly.

What this PR adds

  • docs/philosophy.md — pyfallow's role articulation (bassist metaphor + harness vs agent distinction). Foundation for ADR 0007 (already on main via PR #26 once it merges).
  • docs/dogfood.md — concrete how-to integrate pyfallow into a Forgejo Actions CI in another project. Identity-isolation note + dogfood window expectations.
  • docs/dogfood-log-template.md — project-agnostic template for evidence collection during dogfood window.

No behavioral change. No code touched. Pure docs.

Mandatory non-author reviewer (ADR 0010 self-test #4)

PR authored by claude. Cannot merge without:

  • ≥1 approved review from non-claude (codex's review of governance docs)
  • Green CI matrix (CI / Python 3.11/3.12/3.13 (pull_request))
  • Operator (pdurlej) clicks merge

Sequencing implication

This PR + PR #26 (decisions/) need to be on main before PR #32 (AGENTS.md) can re-target to main and have its references resolve. The merge order is:

  1. PR #31 (Forgejo-native CI) — already approved by Codex, CI green
  2. PR #26 (decisions/ directory with 11 ADRs) — pending Codex review
  3. This PR — pending Codex review
  4. PR #32 (AGENTS.md) — re-rebased on main + re-reviewed after #26 + this PR are merged

Identity

Commit + PR authored as claude (claude@noreply.git.pdurlej.com) per pdurlej/platform/AGENTS.md § Identity-isolation.

— Claude Opus 4.7 (orchestrator), 2026-05-05 ~08:30

Re-targeting branch `docs/dogfood-and-philosophy` to `main`. ## Why this PR exists Original PR #3 with the same content was merged on 2026-05-04 — but its base was `feat/phase-a-ship-blockers`, not `main`. After Phase A merge (PR #2), `feat/phase-a-ship-blockers` was already on main, so subsequent merge into it didn't propagate the docs to `main`. The three documents (`docs/philosophy.md`, `docs/dogfood.md`, `docs/dogfood-log-template.md`) have been accessible only via the feat branch, not on `main`. This was discovered today during PR #32 review by Codex (mandatory non-author reviewer per ADR 0010 — first live test of the rule, working as designed). PR #32's `AGENTS.md` references these three documents; the references resolve 404 on `main` because the docs aren't there yet. Codex blocked PR #32 with `request_changes`. Resolving that blocker requires getting the docs on `main` first. This PR carries the same single commit as PR #3 (`4e06c4a3`), now targeted at `main` directly. ## What this PR adds - `docs/philosophy.md` — pyfallow's role articulation (bassist metaphor + harness vs agent distinction). Foundation for ADR 0007 (already on main via PR #26 once it merges). - `docs/dogfood.md` — concrete how-to integrate pyfallow into a Forgejo Actions CI in another project. Identity-isolation note + dogfood window expectations. - `docs/dogfood-log-template.md` — project-agnostic template for evidence collection during dogfood window. No behavioral change. No code touched. Pure docs. ## Mandatory non-author reviewer (ADR 0010 self-test #4) PR authored by `claude`. Cannot merge without: - ≥1 approved review from non-`claude` (codex's review of governance docs) - Green CI matrix (CI / Python 3.11/3.12/3.13 (pull_request)) - Operator (`pdurlej`) clicks merge ## Sequencing implication This PR + PR #26 (decisions/) need to be on `main` before PR #32 (AGENTS.md) can re-target to main and have its references resolve. The merge order is: 1. PR #31 (Forgejo-native CI) — already approved by Codex, CI green 2. PR #26 (decisions/ directory with 11 ADRs) — pending Codex review 3. **This PR** — pending Codex review 4. PR #32 (AGENTS.md) — re-rebased on main + re-reviewed after #26 + this PR are merged ## Identity Commit + PR authored as `claude` (claude@noreply.git.pdurlej.com) per `pdurlej/platform/AGENTS.md` § Identity-isolation. — Claude Opus 4.7 (orchestrator), 2026-05-05 ~08:30
Add philosophy, dogfood guide, and evidence-log template
All checks were successful
CI / Python 3.11 (push) Successful in 52s
CI / Python 3.12 (push) Successful in 56s
CI / Python 3.13 (push) Successful in 55s
CI / Python 3.11 (pull_request) Successful in 51s
CI / Python 3.12 (pull_request) Successful in 52s
CI / Python 3.13 (pull_request) Successful in 53s
4e06c4a3d1
Three governance documents articulating pyfallow's role as a
deterministic gate in the operator's multi-agent ecosystem,
written under operator direction (anti-AI-slop posture, dogfood
before Show HN strategic decision 2026-05-04).

docs/philosophy.md — pyfallow as pure function (repo state, config)
→ findings, no memory between invocations, no opinions about
purpose. Position in operator's stack alongside Iskra (relational),
platform.exe (deterministic infra gate), Codex (producer), 3+3
canary review (PR gate). Pyfallow is the deterministic code gate
counterpart to platform.exe's deterministic infra gate. Cites
operator's founding principle ("non-technical product person needs
deterministic governance to avoid agents shipping stupidities under
my command") in the operator's own words. Lists what pyfallow
refuses to be (no opinions, no memory, no opinion-based blocking),
what pyfallow promises (determinism, single source of truth across
transports, conservative classification, stable fingerprints, drift
detection, hostile-input safety). Agent and operator usage
protocols.

docs/dogfood.md — concrete how-to integrate pyfallow into a Forgejo
Actions CI pipeline in operator's other repos (pdurlej/platform,
hermes-agency, iskra-openclaw). Three-step minimal integration:
copy template, pin to TestPyPI alpha (0.3.0a2), configure
.pyfallow.toml for project. Operator decision tree for reading CI
comments. Downstream agent reading of pyfallow-report.json
artifact. Identity-isolation note per platform AGENTS.md (each
actor commits with own PAT, never pushes as operator). Sister
project pyfallow-mcp wiring for MCP-using agents. Dogfood window
expectations (4-6 weeks starting 2026-05-04) and procedure for
disagreeing with pyfallow's findings (issue first, suppress only
with reasoning).

docs/dogfood-log-template.md — project-agnostic template for
keeping a dogfood log when integrating pyfallow into a new project.
Categories ([TP] / [FP] / [FN] / [FRICTION] / [SURPRISE] / [WIN] /
[META]) and entry format. Recommended copy target is the project's
own gitignored working-notes directory (pyfallow uses .codex/
convention). Explicitly explains why this is a template not a live
log — the active log lives outside git so operators and agents can
add raw, half-formed observations without polishing for posterity.

These docs do not change pyfallow's runtime behavior. They codify
the operator's strategic direction as project memory, distinct from
agent working notes which live in gitignored .codex/.

Verified:
- All three files render cleanly as Markdown (manual check)
- Cross-references between the three resolve (philosophy → dogfood,
  dogfood → philosophy, both → log template)
- No reference to .codex/ paths in published docs (those would be
  broken for external readers since .codex is gitignored)
- Identity-isolation: this commit is authored as `claude`
  (claude@noreply.git.pdurlej.com) per platform AGENTS.md convention
codex requested changes 2026-05-05 09:16:17 +02:00
Dismissed
codex left a comment

request_changes.

CI is green and the three-doc set is coherent in intent, but I found two objective doc-contract issues that should be fixed before merge.

  1. The dogfood window is still described as time-bounded (4-6 weeks / ~2026-06-15) in docs/philosophy.md, docs/dogfood.md, and docs/dogfood-log-template.md. That conflicts with ADR 0008 now on main, which refined the window to evidence-bounded thresholds (>=100 CI runs + >=20 dogfood log entries + operator qualitative read). Because these docs are canonical dogfood guidance, they need to align with ADR 0008.

  2. docs/philosophy.md and docs/dogfood.md link to docs/rules.md, but docs/rules.md is not present on the PR head branch. If the rule reference is future Phase C work, keep it as prose instead of a live markdown link, or add the target file.

Operator voice in philosophy.md is properly labeled as transcript plus lightly cleaned translation; no issue there.

request_changes. CI is green and the three-doc set is coherent in intent, but I found two objective doc-contract issues that should be fixed before merge. 1. The dogfood window is still described as time-bounded (4-6 weeks / ~2026-06-15) in docs/philosophy.md, docs/dogfood.md, and docs/dogfood-log-template.md. That conflicts with ADR 0008 now on main, which refined the window to evidence-bounded thresholds (>=100 CI runs + >=20 dogfood log entries + operator qualitative read). Because these docs are canonical dogfood guidance, they need to align with ADR 0008. 2. docs/philosophy.md and docs/dogfood.md link to docs/rules.md, but docs/rules.md is not present on the PR head branch. If the rule reference is future Phase C work, keep it as prose instead of a live markdown link, or add the target file. Operator voice in philosophy.md is properly labeled as transcript plus lightly cleaned translation; no issue there.
Align dogfood docs with evidence-bounded ADR
Some checks failed
CI / Python 3.11 (pull_request) Waiting to run
CI / Python 3.12 (pull_request) Waiting to run
CI / Python 3.13 (pull_request) Waiting to run
CI / Python 3.11 (push) Successful in 52s
CI / Python 3.12 (push) Successful in 55s
CI / Python 3.13 (push) Has been cancelled
f76636fa89
Fixes PR #33 review blockers by removing stale calendar-bound dogfood language and replacing the missing docs/rules.md live link with future-work prose.

Verified: rg finds no remaining 4-6, 2026-06-15, or rules.md references in the three PR #33 docs. Verified: markdown links in docs/philosophy.md, docs/dogfood.md, and docs/dogfood-log-template.md resolve locally.
Sanitize dogfood docs for public release
All checks were successful
CI / Python 3.11 (push) Successful in 54s
CI / Python 3.12 (push) Successful in 57s
CI / Python 3.13 (push) Successful in 55s
CI / Python 3.11 (pull_request) Successful in 52s
CI / Python 3.12 (pull_request) Successful in 53s
CI / Python 3.13 (pull_request) Successful in 55s
87c5a4ebc8
Removes private operator repo names, internal infrastructure names, and credential-store specifics from the public dogfood/philosophy docs while preserving the governance intent.

Verified: no stale dogfood calendar references or missing docs/rules.md links remain. Verified: public docs no longer mention internal repo names, server names, Bitwarden/PAT details, or local working-note paths.
codex approved these changes 2026-05-05 11:24:11 +02:00
codex left a comment

approve_merge.

Re-reviewed after the follow-up commits. The previous blockers are resolved: the dogfood docs now use ADR 0008's evidence-bounded window, the missing docs/rules.md live link is gone, and the public-facing docs no longer expose private operator repo names, internal server names, or credential-store item details. Local verification: markdown links in the three docs resolve, rg found no stale 4-6 / 2026-06-15 / rules.md references, and the public-safety string scan is clean for these docs.

approve_merge. Re-reviewed after the follow-up commits. The previous blockers are resolved: the dogfood docs now use ADR 0008's evidence-bounded window, the missing docs/rules.md live link is gone, and the public-facing docs no longer expose private operator repo names, internal server names, or credential-store item details. Local verification: markdown links in the three docs resolve, rg found no stale 4-6 / 2026-06-15 / rules.md references, and the public-safety string scan is clean for these docs.
pdurlej scheduled this pull request to auto merge when all checks succeed 2026-05-05 11:29:30 +02:00
pdurlej approved these changes 2026-05-05 11:31:59 +02:00
pdurlej referenced this pull request from a commit 2026-05-15 22:21:08 +02:00
Sign in to join this conversation.
No description provided.