Skip to content

Document private field searchability default behavior - #3440

Draft
pwizla wants to merge 1 commit into
mainfrom
cms/private-field-searchability-default
Draft

Document private field searchability default behavior#3440
pwizla wants to merge 1 commit into
mainfrom
cms/private-field-searchability-default

Conversation

@pwizla

@pwizla pwizla commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

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

The Content-Type Builder now sets newly created private fields to be non-searchable by default (searchable: false). This prevents private content from appearing in _q search results. This documentation update explains this behavior and clarifies that existing private fields keep their current searchability settings.

Generated automatically by the docs self-healing workflow (micro-edit, Haiku).
Review before merging.

@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 3, 2026
@pwizla pwizla self-assigned this Sep 3, 2026
@strapi-cla

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 sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel

vercel Bot commented Sep 3, 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 3, 2026 10:10am UTC

Request Review

Add documentation explaining that newly created private fields default to non-searchable (searchable: false) to prevent private content from appearing in _q search results. Existing private fields retain their current searchability settings.

Relates to strapi/strapi#27482
@pwizla
pwizla force-pushed the cms/private-field-searchability-default branch from 4b29a84 to 1a3310b Compare September 3, 2026 10:09
@pwizla pwizla added the automerge: eligible can be auto-merged (handled by automerge-cron.yml); tag added by automerge-eligibility.yml label Sep 3, 2026
pwizla added a commit that referenced this pull request Sep 3, 2026
…rting (#3441)

* Reroute misclassified micro targets to Sonnet instead of only warning

Whether a doc target is `micro` or `full` is a mechanical rule: micro means
every target action is add_link, add_mention or add_tip. Haiku has now got it
wrong twice, in opposite directions, and both times the run ended green.

On 2026-06-20 (run 27857268241) it stamped `micro` on update_section targets
and then did nothing, so FULL=0 gated Sonnet out and both PRs were lost. On
2026-09-03 (run 33705096766) it stamped `micro` on add_section and
update_section targets and drafted whole sections itself, prose its micro-edit
procedure does not even cover, and Sonnet was again never invoked.

Apply the rule in the workflow rather than asking the model to honour it. A
misclassified target with no doc PR is rewritten to `full` so Sonnet picks it
up; one that already has a doc PR is left alone, since rerouting would
duplicate the PR, and flagged as drafted beyond the micro scope.

Report the three outcomes separately. Sharing one message is what made the
2026-09-03 Slack post announce "2 doc PR(s) created" and "claimed but not
delivered" about the very same two PRs.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* Filter the Drafter payload to full targets without a doc PR

The step comment claimed it passed "only full-complexity PRs" to Sonnet while
handing over the whole router results file, and the Drafter prompt filters on
`decision` alone. A run with both micro and full targets would therefore have
had Sonnet redraft the micro PRs Haiku had just opened.

Filter here rather than trusting the prompt, and exclude anything that already
carries a doc PR URL.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* Make the self-healing Router branch from a clean main

Resetting to origin/main at the end of each micro-edit iteration was not
enough. On 2026-09-03 the Router opened PR #3439, then ran `git checkout -b`
for the next PR while still on that branch, so PR #3440 carried #3439's commit
as well and touched a file unrelated to its own source PR.

Reset before creating the branch, so each branch is independent of whatever the
previous iteration left behind.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* Forbid the self-healing Router from drafting sections

The complexity rule was stated once, as a definition, and Haiku twice decided
against it after reasoning correctly about the same targets in prose. Say
plainly that the classification comes from the action values alone, that one
add_section among ten add_link targets still makes the PR full, and that the
workflow now overwrites a wrong answer.

Add the matching hard stop to the rules list: a micro-edit is a link, a mention
or a tip, and anything needing a section stops at the routing decision.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@pwizla pwizla added the automerge: approved by piwi Maintainer vouched for this PR: skip the mechanical eligibility checks and merge it label Sep 3, 2026
@pwizla

pwizla commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

LGTM, just waiting for the corresponding strapi/strapi PR to be released

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 automerge: approved by piwi Maintainer vouched for this PR: skip the mechanical eligibility checks and merge it automerge: eligible can be auto-merged (handled by automerge-cron.yml); tag added by automerge-eligibility.yml flag: merge pending release Waiting for the related product PR to be merged before merging this 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