Skip to content

Document release actions in audit logs - #3443

Open
pwizla wants to merge 2 commits into
mainfrom
cms/document-release-audit-logs
Open

Document release actions in audit logs#3443
pwizla wants to merge 2 commits into
mainfrom
cms/document-release-audit-logs

Conversation

@pwizla

@pwizla pwizla commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

This PR updates documentation based on strapi/strapi#27436.

Generated automatically by the docs self-healing workflow.
Review before merging.

Add release events (create, update, delete, trigger, entry add/change/remove,
settings update) to the audit-logs event table, and add a dedicated section
to the releases page explaining what is recorded and how to view it.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pwizla pwizla added flag: merge pending release Waiting for the related product PR to be merged before merging this documentation PR auto-doc-healing Automatically generated documentation PR labels Sep 4, 2026
@pwizla pwizla self-assigned this Sep 4, 2026
@strapi-cla

strapi-cla commented Sep 4, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ pwizla
❌ github-actions[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
documentation Ready Ready Preview Sep 4, 2026 7:49am UTC

Request Review

Comment thread docusaurus/docs/cms/features/releases.md Outdated
@pwizla
pwizla marked this pull request as ready for review September 4, 2026 07:48

@pwizla pwizla left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM, mergeable when ready 🚀

pwizla added a commit that referenced this pull request Sep 4, 2026
The Drafter prompt already forbade em dashes and PR #3443 carried one anyway: the rule sits 300 lines before the first sentence is written, so it now runs again as a sweep at output time. Micro-edits had no rule at all, since that path loads router.md only and router.md never mentioned em dashes.
pwizla added a commit that referenced this pull request Sep 4, 2026
…#3444)

* Add an em dash check to the auto-merge eligibility criteria

Em dashes are not used in Strapi documentation and are a strong tell of unedited AI-generated prose. The self-healing workflow now lints them out before committing, and this check is the net under that: it reports, it never edits the PR.

* Run style-lint before committing in the self-healing workflow

The Style Checker prompt already forbids em dashes, but a prompt is a probabilistic filter and style-lint.sh is not: an em dash still reached PR #3443. Micro-edits skip the Style Checker entirely, so the script is their only style guard. The Router block is reordered first, since its git reset ran after the file edits and would have destroyed them.

* Rewrite the self-healing PR descriptions to follow git-rules.md

The canned body named no change, put the source PR as a bare URL instead of a Documents link, carried no preview link, and repeated boilerplate the label, assignee and draft status already convey. Both prompts now build the description from the actual diff. Also repairs the broken line continuations in the Drafter gh pr create call, which would have dropped the body, labels and draft flag.

* Never let the style-lint pass stop the self-healing workflow from opening a PR

A remaining em dash costs a few seconds of editing at review time. An abandoned PR costs re-reading the strapi/strapi diff and writing the page by hand, and the source PR does not come back: the next run only lists PRs merged in the last 24 hours, so anything skipped is silently lost.

* State the em dash rule where the agent actually reads it

The Drafter prompt already forbade em dashes and PR #3443 carried one anyway: the rule sits 300 lines before the first sentence is written, so it now runs again as a sweep at output time. Micro-edits had no rule at all, since that path loads router.md only and router.md never mentioned em dashes.
@pwizla
pwizla requested a review from gonbaum September 8, 2026 13:08
@pwizla

pwizla commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator Author

@gonbaum The self-healing system created this docs PR to document the strapi/strapi PR you created.
Sounds good to you? 👀

@pwizla pwizla left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Approved from a tech. writing POV ✅

@pwizla pwizla removed the flag: merge pending release Waiting for the related product PR to be merged before merging this documentation PR label Sep 9, 2026
@pwizla pwizla added this to the 7.2.0 milestone Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-doc-healing Automatically generated documentation PR internal PRs created by the Strapi core team pr: chore source: CMS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants