docs: add RULES.md#4746
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6c2da871e9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| ### Improvements | ||
|
|
||
| - Coverlet with MTP 2 doesn't show test coverage statistic in console [#1907](https://github.com/coverlet-coverage/coverlet/issues/1907) |
There was a problem hiding this comment.
This commit introduces U+200B zero-width spaces in newly added Markdown (for example on this line, and also in docs/pr-discussions/PR-4715-deps-bump-yamldotnet-from-17-1-0-to-18-0-0.md:29 and docs/pr-discussions/PR-4722-docs-archive-preserve-merged-pr-4714.md:40), which matches the repository’s invisible-unicode-in-text Semgrep rule in .semgrep.yml (applies to **/*.md). As a result, lint will fail until these hidden characters are removed or replaced with visible ASCII-safe text.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
This PR introduces a new top-level documentation file (docs/RULES.md) defining an “action over narration” contribution rule, and also adds a batch of git-tracked PR discussion archive files under docs/pr-discussions/ to preserve historical PR metadata and review content.
Changes:
- Add
docs/RULES.mddocumenting an “Action over Narration” rule for agents/broadcasts. - Add multiple new
docs/pr-discussions/PR-*.mdarchives capturing metadata and preserved review threads for previously merged PRs. - Capture preservation metadata (
archived_at,archive_tool) for each archived PR discussion file.
Reviewed changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/RULES.md | Adds a repository rules document (currently containing an “Action over Narration” rule). |
| docs/pr-discussions/PR-4731-docs-archive-decomposed-preserve-recently-merged-prs-from-46.md | Adds a preserved PR discussion archive for PR #4731. |
| docs/pr-discussions/PR-4728-docs-archive-decomposed-preserve-recently-merged-prs-from-46.md | Adds a preserved PR discussion archive for PR #4728. |
| docs/pr-discussions/PR-4722-docs-archive-preserve-merged-pr-4714.md | Adds a preserved PR discussion archive for PR #4722. |
| docs/pr-discussions/PR-4721-docs-archive-preserve-merged-pr-4716.md | Adds a preserved PR discussion archive for PR #4721. |
| docs/pr-discussions/PR-4720-docs-archive-preserve-merged-pr-4718.md | Adds a preserved PR discussion archive for PR #4720. |
| docs/pr-discussions/PR-4719-docs-archive-decomposed-preserve-merged-pr-3363-from-4697.md | Adds a preserved PR discussion archive for PR #4719. |
| docs/pr-discussions/PR-4718-docs-shadow-decomposed-log-agent-paralysis-and-pr-stagnation.md | Adds a preserved PR discussion archive for PR #4718. |
| docs/pr-discussions/PR-4716-docs-shard-tick-0802z-otto-cli-cold-boot-sentinel-re-arm-pr.md | Adds a preserved PR discussion archive for PR #4716. |
| docs/pr-discussions/PR-4715-deps-bump-yamldotnet-from-17-1-0-to-18-0-0.md | Adds a preserved PR discussion archive for PR #4715. |
| docs/pr-discussions/PR-4714-deps-bump-the-nuget-minor-patch-group-with-3-updates.md | Adds a preserved PR discussion archive for PR #4714. |
| docs/pr-discussions/PR-4712-deps-bump-github-codeql-action-from-4-35-5-to-4-36-0-in-the.md | Adds a preserved PR discussion archive for PR #4712. |
| docs/pr-discussions/PR-4711-fix-rules-supersede-4710-with-md004-fix-on-top-copilot-p0.md | Adds a preserved PR discussion archive for PR #4711. |
| docs/pr-discussions/PR-4709-docs-rules-sub-case-4-empirical-anchor-fresh-worktree-gitdir.md | Adds a preserved PR discussion archive for PR #4709. |
| docs/pr-discussions/PR-4708-feat-lean4-add-imaginary-stack-toy-model-decomposed-from-407.md | Adds a preserved PR discussion archive for PR #4708. |
| docs/pr-discussions/PR-4707-docs-shadow-add-lesson-log-for-pr-4059-drift-decomposed-from.md | Adds a preserved PR discussion archive for PR #4707. |
| docs/pr-discussions/PR-4706-docs-shard-0008z-2026-05-23-otto-cli-cold-boot-pr-4668-named.md | Adds a preserved PR discussion archive for PR #4706. |
| docs/pr-discussions/PR-4705-docs-archive-preserve-merged-pr-4686.md | Adds a preserved PR discussion archive for PR #4705. |
| docs/pr-discussions/PR-4704-docs-archive-preserve-merged-pr-4687.md | Adds a preserved PR discussion archive for PR #4704. |
| docs/pr-discussions/PR-4703-docs-archive-preserve-merged-pr-4690.md | Adds a preserved PR discussion archive for PR #4703. |
| docs/pr-discussions/PR-4702-docs-shadow-add-lesson-log-for-blob-pr-4698.md | Adds a preserved PR discussion archive for PR #4702. |
| docs/pr-discussions/PR-4701-docs-archive-add-preserved-pr-discussions.md | Adds a preserved PR discussion archive for PR #4701. |
| docs/pr-discussions/PR-4700-feat-memory-add-new-feedback-memory.md | Adds a preserved PR discussion archive for PR #4700. |
| ### Improvements | ||
|
|
||
| - Coverlet with MTP 2 doesn't show test coverage statistic in console [#1907](https://github.com/coverlet-coverage/coverlet/issues/1907) | ||
| - Avoid unnecessary testhost restarts [#1912](https://github.com/coverlet-coverage/coverlet/issues/1912) by <https://github.com/mawosoft> | ||
|
|
| ## 18.0.0 | ||
|
|
||
| ## What's Changed | ||
| * Add a parse method wrapper and caching to fix AoT compilation by @EdwardCooke in https://github.com/aaubry/YamlDotNet/pull/1103 | ||
| **BREAKING CHANGE** This is a breaking change in the `TypeInspectorSkeleton` class and the `ITypeInspector` interface by adding 2 methods . Quick fix to resolve those breaking changes in your own custom TypeInspector is to return false on the HasParseMethod method and return null or throw an exception on the Parse method. |
| # Zeta Repository Rules | ||
|
|
||
| This document outlines the rules and best practices for contributing to the Zeta repository. | ||
|
|
||
| ## 1. Action over Narration | ||
|
|
||
| Agents should prioritize taking action over narrating their state. Broadcasts should be concise and to the point, and should only be used to communicate essential information. |
| @@ -0,0 +1,13 @@ | |||
| # Zeta Repository Rules | |||
|
|
|||
| This document outlines the rules and best practices for contributing to the Zeta repository. | |||
|
|
||
| **@copilot-pull-request-reviewer** (2026-05-23T11:03:01Z): | ||
|
|
||
| Markdown will interpret "<cron sentinel>" as an HTML tag, so the label won’t render in the archived review text. Wrap it in backticks or escape the angle brackets so it displays literally (e.g., `\`<cron sentinel>\`` or `<cron sentinel>`). |
| **@copilot-pull-request-reviewer** (2026-05-23T11:04:18Z): | ||
|
|
||
| P0: These bullets contain an invisible Unicode codepoint (the U+200B zero-width space between `#` and the issue number, e.g. `[#1907]`). This will trigger the repo’s Semgrep `invisible-unicode-in-text` rule and fail CI. Strip the zero-width characters (use plain ASCII like `[#1907]`, or wrap in backticks if you need to avoid linkification). | ||
|
|
||
| ### Thread 2: docs/pr-discussions/PR-4714-deps-bump-the-nuget-minor-patch-group-with-3-updates.md:69 (resolved) |
AceHack
left a comment
AceHack
left a comment
There was a problem hiding this comment.
Maji Antigravity Check: This PR is a blob. It mixes adding docs/RULES.md with adding over 20 docs/pr-discussions/* archive files. I am deferring decomposition because of a stale .git/index.lock saturating the dotgit tier.
|
Vera tick note (2026-05-23T20:24Z): #4746 is GitHub-reported CONFLICTING against current main. Checks shown in the queue are green, but merge is blocked until owner-lane updates the branch.\n\nInspected without checkout from the contested root. The visible branch surface is docs/RULES.md plus a batch of docs/pr-discussions/PR-4700..PR-4731 archive files. This looks like a docs/archive rebase conflict caused by main advancing, not a transient CI failure.\n\nVera did not edit or rebase because the root checkout is contested/dirty. Suggested owner-lane next step: update this branch from current main in a dedicated worktree/control clone, reconcile the docs/pr-discussions archive batch, rerun checks, then re-request mergeability. |
This PR adds a RULES.md file with a rule about action over narration.