Skip to content

docs: 'Choosing the Right Options' feature-discovery guide#204

Merged
bosd merged 2 commits into
masterfrom
docs/feature-overview
Jun 11, 2026
Merged

docs: 'Choosing the Right Options' feature-discovery guide#204
bosd merged 2 commits into
masterfrom
docs/feature-overview

Conversation

@bosd

@bosd bosd commented Jun 11, 2026

Copy link
Copy Markdown
Member

Output of the docs/discoverability pass.

The gap

A doc audit found that while the CLI reference is complete (sphinx-click auto-documents every flag's --help), there's no conceptual entry point: a user with a problem ("my import is slow", "I'm hitting deadlocks", "my source data is messy") has to scroll ~40 flags to find the right one. The powerful features (--resolve-relation, --skip-unchanged, --adaptive-throttle, --check-refs, --auto-clean, ...) were discoverable only by reading the full reference.

The fix

A new "Choosing the Right Options" guide — a problem→feature map grouped by intent:

  • "My import is slow" / "relational data" / "re-running (idempotency)" / "unstable server or huge import" / "messy source data" / "concurrent-update deadlocks" / "hierarchical data" / product-specific

Each row names the flag + what it does, linking to the in-depth guides and the CLI reference. Wired early in the guides toctree as a discovery aid. Builds clean (0 new warnings).

Note: links target the guide files (not section anchors) because myst_heading_anchors isn't enabled — robust and warning-free.

The CLI reference (sphinx-click) documents every flag, but a user with a *problem*
can't easily tell which feature solves it. Add a problem->feature map covering
speed, relational data, idempotency, resilience, messy data, deadlocks, and
hierarchies, linking to the in-depth guides. Wired early in the guides toctree.

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request introduces a new guide, choosing_options.md, which maps common import problems to specific fluvo features and links to relevant in-depth guides. It also updates the guides index to include this new document. Feedback was provided to use HTML entities (|) instead of backslash-escaped pipes inside the Markdown table to ensure consistent rendering across different Markdown engines.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread docs/guides/choosing_options.md Outdated
|---|---|
| (automatic two-pass) | Relational fields are deferred to a second pass to avoid inverse-relation rewrites ([Performance Tuning](performance_tuning.md)) |
| `--deferred-fields` / `--auto-defer` | Manually or automatically defer specific relational fields |
| `--check-refs fail\|warn\|skip` | Pre-import validation that referenced XML IDs exist |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

In Markdown tables, using a backslash to escape a pipe character (\|) inside inline code can sometimes render the backslash literally (as \|) or cause parsing issues depending on the Markdown engine.\n\nUsing the HTML entity | is the standard, most portable way to represent a pipe character inside a table cell without breaking the table structure or rendering an unwanted backslash.

Suggested change
| `--check-refs fail\|warn\|skip` | Pre-import validation that referenced XML IDs exist |
| `--check-refs fail|warn|skip` | Pre-import validation that referenced XML IDs exist |

Backslash-escaped pipes (\|) can render literally or break tables on some
markdown engines; the HTML entity is portable.
@bosd

bosd commented Jun 11, 2026

Copy link
Copy Markdown
Member Author

Done — replaced the two \| escaped pipes with | for portable table rendering. Builds clean.

@bosd bosd merged commit 1dfb289 into master Jun 11, 2026
24 checks passed
@bosd bosd deleted the docs/feature-overview branch June 11, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant