docs(sales): the Products panel adds a line item, in all three locales

[os-bill]

Fixes #1758

#1720 documented yesterday's behaviour honestly — nothing on the opportunity screen adds a line item. #1757 made that false the same day by giving opp_products an add picker. This corrects the two passages in all three locales without discarding the four claims that are still true.

What changed

Three docs files, two lines each, plus one changeset. Passages were re-located by content; the card's line numbers were as of 456a0a76 and still happen to resolve to 124 and 184 on d47e37ac.

file	line 124 — Line items — what's actually being sold	line 184 — Related tab bullet
content/docs/sales/opportunities.mdx	rewritten	Products clause rewritten
content/docs/sales/opportunities.zh-Hans.mdx	rewritten	Products clause rewritten
content/docs/sales/opportunities.zh-Hant.mdx	rewritten	Products clause rewritten

.changeset/opportunities-page-catches-up-with-the-add-picker.md — added.

No src/ change. No content/docs/releases/ change. No new gate, guard, lint rule or test.

Written from the distinction the page had already earned

The page already taught the reader that a related list gets a button in exactly two ways, an actions list or an add picker. The app now uses the second and still not the first, so that is how the correction is phrased — not "an Add Product action was added" (false), and not a vague "you can now add products" (throws away the precision).

Seven claims on line 124; three flipped, four survive and are still on the page:

#	claim	state	in the new prose
1	"Nothing on the opportunity screen adds one"	flipped	"You add one from the Products panel"
2	a button comes in exactly two ways, actions or add	kept	kept verbatim, and now load-bearing
3	"this one declares neither"	flipped	"declares the second and still not the first"
4	no Add Product action anywhere in the app	kept	kept, both lines
5	crm_opportunity_line_item has no list view of its own	kept	kept verbatim
6	permission is not what is missing	kept	kept, retensed
7	typing the Amount by hand is a supported route, not a workaround	kept	kept verbatim

Verified per locale rather than assumed: all seven claims were present in each of the three files, and each survives or flips identically in each.

What the pages gained

From #1757's own measurements: the picker offers active products only (is_active), so a retired product cannot be sold onto a new deal; and the row it creates arrives complete and priced — quantity 1, the product's list price as the unit price, no discount — for the rep to adjust. Rows read as the catalog names a product, PRD-0001 - Widget. Import and the API are described as still working, simply no longer the only route.

No timing promise is made about the opportunity Amount. opportunity_amount_rollup is async: true with onError: 'log', so the roll-up is not in the same transaction. Describing the lag would be a different card; the fix here is to say nothing about timing.

The button is not given a Chinese name — measured, not assumed

The picker deliberately authors no add.label, so the label is the platform's. Measured in the pinned @objectstack/console build rather than guessed:

• The renderer resolves it as w.label || W('detail.add', {defaultValue: 'Add'}) — plugins-views-iYcMmImr.js.
• detail.add is Add in en-US and 添加 in zh-CN — i18n-locales-Bz4c8KXc.js.
• That bundle ships en-US, zh-CN, ja-JP, es-ES. There is no zh-Hant / zh-TW locale at all, and objectstack.config.ts supportedLocales agrees.

So a Traditional rendering does not exist to source, and the Simplified one lives in a platform build artifact this repo does not own, with no gate to catch it drifting on a platform bump. Rather than coin one, both Chinese pages name the button as the surrounding prose already names View all and Products, and add that its label follows the reader's interface language — which is true in every locale and cannot go stale.

Where this repo does own the wording, the pack is used: "active" is written 已启用 / 已啟用 against crm_product.is_active → 是否启用 in src/translations/zh-CN/objects.commerce.ts.

Verification

pnpm verify = validate && typecheck && lint && lint:i18n-gate && hygiene && hygiene:tokens && build && test — exit code 0, run on the committed tree. Test Files 164 passed (164), Tests 3438 passed | 1 skipped (3439). The 5 author-time build warnings are pre-existing on main and unrelated.

Checks that would not have caught this, stated because it is why the card exists:

• test/docs-drift.test.ts names all three files but pins the large-deal amount condition (capCel('opportunity_won_alert', ...)), not this prose. Swept test/ — no test pins any of the six passages.
• link-check.yml triggers on .mdx but scans .md exact, so on this diff it opens only the changeset. Checked by hand instead: this change introduces no links, and the one link inside the edited line 184 (Quote Generation flow) is untouched.

Deliberately no guard added — pinning prose against page metadata is how a sibling check went vacuous (#1755), and a guard that would be vacuous is worse than none.

Byte discipline: grep -naP '[\x00-\x08\x0b\x0c\x0e-\x1f\x7f]' over all three files returns no matches. Script-leakage check confirms no Simplified forms in the zh-Hant lines and no Traditional forms in the zh-Hans lines.

🤖 Generated with Claude Code

https://claude.ai/code/session_019YKN9TPqDSSXUaYqp2j7MZ

---

Generated by Claude Code

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
hotcrm	Ignored		Sep 8, 2026 5:34am UTC