Skip to content

docs(agents): add CWL-ENTRY read-first block - #1528

Draft
seonghobae wants to merge 6 commits into
autoresearch/frontend-sec-bumpfrom
docs/agents-add-cwl-entry-block
Draft

docs(agents): add CWL-ENTRY read-first block#1528
seonghobae wants to merge 6 commits into
autoresearch/frontend-sec-bumpfrom
docs/agents-add-cwl-entry-block

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Current authority — 2026-09-10

  • base owner: #1623@17a7618eda2b212b691f08fa936e042b34258fc9
  • exact head: 2bcf5eb8b93b231c05f5f04d6a2a31190cf7df81
  • lifecycle: Draft / repository-entry guidance owner / do not merge
  • effective delta over current fix(deps): patch frontend audit security floors #1623: exactly 2 filesAGENTS.md and backend/tests/test_release_governance.py.

Current head is an ordinary two-parent successor: old #1528 83b302a... remains first parent and current #1623 is the second parent. The tree adopts #1623's latest Vitest importer-contract repair while preserving the repository-entry guidance delta; dependency/security source remains owned by #1623.

Fresh compare against exact base 17a7618... is ahead-only with the same two-file product/governance scope. No force push, destructive rebase, copied dependency source, or predecessor-evidence transfer was used.

This source-changing restack has no ordinary PR-triggered hosted workflow run at the latest read, so earlier checks/reviews are historical only. Keep Draft until the prerequisite is protected-integrated and this exact head has then-live required evidence plus qualifying independent review.

Governance descendant #1549 has been ordinary-restacked onto this head as 1c343aaaf080bdd009ba9784de721a1c26d4cdc5.

.github's AGENTS.md carries a <!-- CWL-ENTRY --> block pointing agents at
the master context, live GitHub Project, and gap snapshot before any work.
naruon never had this block (it only has a naruon-specific
cwl-agent-guidance section below it), so agents landing here had no
equivalent pointer. Add naruon's own version, wired to naruon's real docs:
docs/architecture/naruon-product-spec.md for mission/architecture, the
existing docs/product-technical-gap-baseline.md gap snapshot, and a
cross-repo link to .github's agent-github-project-protocol.md and
product-goal-directive.md, which are genuinely org-wide and not duplicated
per repo. The Figma file ID line reflects naruon's actual current state
(not yet canonical on develop; ADR-0013 is still in open PR #1436) rather
than the N/A placeholder that would make no sense for a repo that has UI.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@seonghobae seonghobae added documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: docs Documentation, ADR, PRD, or technical writing labels Sep 2, 2026 — with ChatGPT Codex Connector
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

PR governance metadata gate is not ready for cf472cf77fb93325858f485a22e967449d7c387a:

  • Required check strix is CANCELLED on the current head.

seonghobae added a commit to ContextualWisdomLab/.github that referenced this pull request Sep 3, 2026
…dict contradiction

- Cite ContextualWisdomLab/naruon#1528 with owner prefix in both remaining
  spots (Devin Review: cross-repo reference format violation)
- Fix the self-contradiction Devin caught: "Result 1 verified all four
  central workflows' cancellation logic is correct" directly contradicted
  the same document's confirmed noema-review.yml bug finding above it.
  Corrected to: three of four are fully correct; the fourth's confirmed
  bug has a different failure signature (wrongful cancellation of a live
  run) than what Result 2's evidence shows (runs stuck queued for
  23h22m/24+ hours with no cancellation at all) -- so the bug still
  doesn't explain this specific wait, without overstating the workflows'
  correctness.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Format the CWL entry as a readable source checklist and add a governance regression for the product spec, Project, gap baseline, completion issue, Figma proposal, and central operating documents.\n\nAssisted-by: OpenAI Codex <codex@openai.com>
@seonghobae
seonghobae marked this pull request as draft September 4, 2026 20:51
seonghobae added a commit that referenced this pull request Sep 4, 2026
Record exact heads and verified successor boundaries for PRs #1528, #1549, #1566, and #1567.\n\nAssisted-by: OpenAI Codex <codex@openai.com>
@seonghobae seonghobae removed the status: needs-review Open pull request requiring current-head review or checks label Sep 6, 2026
@seonghobae seonghobae added the status: draft Draft pull request label Sep 6, 2026 — with ChatGPT Codex Connector
@seonghobae
seonghobae marked this pull request as ready for review September 7, 2026 12:46
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review the unchanged exact head 57254cb61ecc67dd593ea96ae878bf651682a501 against protected develop@042b0c70531b229af3acbd0421a2f23098d848b3. The effective delta is only AGENTS.md and backend/tests/test_release_governance.py. Treat historical cancelled checks and comment-only review as non-authoritative; verify current source and report any still-valid finding.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

base="042b0c70531b229af3acbd0421a2f23098d848b3"
head="57254cb61ecc67dd593ea96ae878bf651682a501"

printf '%s\n' '== Checked-out commit and requested commit objects =='
git rev-parse HEAD
git cat-file -t "$base"
git cat-file -t "$head"
git merge-base --is-ancestor "$base" "$head" && echo "base_is_ancestor=true"

printf '%s\n' '== Name/status and complete exact diff =='
git diff --check "$base" "$head"
git diff --name-status "$base" "$head"
git diff --find-renames --find-copies --unified=80 "$base" "$head" -- AGENTS.md backend/tests/test_release_governance.py

printf '%s\n' '== AGENTS.md CWL entry context =='
rg -n -C 25 '<!-- CWL-ENTRY -->|CWL-ENTRY|ContextualWisdomLab/\.github|Project|Figma|`#1428`|`#1436`' AGENTS.md || true

printf '%s\n' '== Governance test structure and changed-entry references =='
ast-grep outline backend/tests/test_release_governance.py --items all --view expanded
rg -n -C 12 'CWL-ENTRY|AGENTS\.md|1428|1436|canonical|Project|Figma|private agent memory|stale repository' backend/tests/test_release_governance.py || true

printf '%s\n' '== Current pull request metadata, checks, and reviews =='
gh pr view 1528 --repo ContextualWisdomLab/naruon \
  --json number,url,state,isDraft,headRefOid,baseRefOid,mergeStateStatus,reviewDecision,statusCheckRollup,reviews \
  --jq '{number,url,state,isDraft,headRefOid,baseRefOid,mergeStateStatus,reviewDecision,statusCheckRollup,reviews}'

Length of output: 47991


🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '%s\n' '== Referenced local source files =='
for path in \
  docs/architecture/naruon-product-spec.md \
  docs/product-technical-gap-baseline.md \
  backend/tests/test_release_governance.py \
  AGENTS.md
do
  if test -f "$path"; then
    printf 'present: %s\n' "$path"
  else
    printf 'missing: %s\n' "$path"
  fi
done

printf '%s\n' '== CWL entry block containment and duplicate reference locations =='
python3 - <<'PY'
from pathlib import Path

agents = Path("AGENTS.md").read_text(encoding="utf-8")
start = agents.index("<!-- CWL-ENTRY -->")
end = agents.index("<!-- BEGIN cwl-agent-guidance -->", start)
entry = agents[start:end]
needles = (
    "docs/architecture/naruon-product-spec.md",
    "docs/product-technical-gap-baseline.md",
    "ContextualWisdomLab/projects/1",
    "ContextualWisdomLab/naruon/issues/1428",
    "ContextualWisdomLab/naruon/pull/1436",
    "docs/agent-github-project-protocol.md",
    "docs/product-goal-directive.md",
    "not private agent memory",
)
for needle in needles:
    positions = []
    offset = 0
    while True:
        index = agents.find(needle, offset)
        if index < 0:
            break
        positions.append(index)
        offset = index + len(needle)
    print(f"{needle}: total={len(positions)}, in_entry={entry.count(needle)}")
PY

printf '%s\n' '== Test guard shape =='
sed -n '1,70p' backend/tests/test_release_governance.py
sed -n '600,620p' backend/tests/test_release_governance.py

printf '%s\n' '== Live authority records =='
gh issue view 1428 --repo ContextualWisdomLab/naruon \
  --json number,state,title,url,closedAt \
  --jq '{number,state,title,url,closedAt}'
gh pr view 1436 --repo ContextualWisdomLab/naruon \
  --json number,state,isDraft,title,url,headRefOid,baseRefName \
  --jq '{number,state,isDraft,title,url,headRefOid,baseRefName}'

Length of output: 6622


@seonghobae: I found one still-valid finding.

P2 — Make the regression test validate the CWL-ENTRY block and link destinations.
backend/tests/test_release_governance.py, Line 607 through Line 615, checks only whether each required string occurs somewhere in the full AGENTS.md text. The test passes if a future edit removes the references from <!-- CWL-ENTRY -->, retains them elsewhere in the document, or changes a Markdown link destination while keeping the current visible label. It also does not reject a stale competing Project, issue, PR, or Figma reference in the entry block.

Extract the text from <!-- CWL-ENTRY --> through <!-- BEGIN cwl-agent-guidance -->. Then assert the required Markdown destinations and authority statement within that block. Add negative assertions for superseded authority references when their values are known.

The exact head and effective two-file delta match the request. git diff --check passed. The local product specification and gap-baseline files exist. Issue #1428 is open. PR #1436 is open and Draft.

The current GitHub state is still not merge-authoritative. The pull request is BLOCKED and REVIEW_REQUIRED. The listed required workflow runs are cancelled or skipped. The listed review is comment-only and targets a different commit.

You are interacting with an AI system.

@seonghobae
seonghobae changed the base branch from develop to autoresearch/frontend-sec-bump September 9, 2026 04:43
Preserve the repository-entry AGENTS/test delta while ordinary-adopting #1623's js-yaml 4.3.2 and Vitest 4.1.11 security head. No dependency files move into this documentation prerequisite.
seonghobae added a commit that referenced this pull request Sep 9, 2026
Preserve the canonical five-file LLM-governance delta while ordinary-adopting #1528 after its #1623 js-yaml/Vitest security restack. Dependency source remains owned by #1623.
seonghobae added a commit that referenced this pull request Sep 9, 2026
…head

Preserve the five-file central OpenCode consumer delta while ordinary-adopting #1549 after its current #1528/#1623 prerequisite restack. The local review config remains deleted and no central workflow source is copied into Naruon.
Preserve the repository-entry two-file delta as first-parent history while adopting #1623's current Vitest lock-presence security contract.

Signed-off-by: Seongho Bae <me@seonghobae.me>
seonghobae added a commit that referenced this pull request Sep 9, 2026
Preserve the five-file governance delta as first-parent history while adopting #1528's current security ancestry. Governance ownership remains on this lane; dependency source remains on #1623.

Signed-off-by: Seongho Bae <me@seonghobae.me>
Preserve the repository-entry guidance delta as first-parent history while adopting #1623's current Vitest importer-contract repair.

Signed-off-by: Seongho Bae <me@seonghobae.me>
seonghobae added a commit that referenced this pull request Sep 9, 2026
Preserve the five-file Naruon governance delta as first-parent history while adopting #1528's current #1623 security ancestry.

Signed-off-by: Seongho Bae <me@seonghobae.me>

Copy link
Copy Markdown
Contributor Author

@codex review

Review only exact head 2bcf5eb8b93b231c05f5f04d6a2a31190cf7df81 against base 17a7618eda2b212b691f08fa936e042b34258fc9. Verify the two-file repository-entry guidance delta and its governance regression test; treat all predecessor reviews/checks as historical after the ordinary restack.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: draft Draft pull request type: docs Documentation, ADR, PRD, or technical writing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant