Conversation
Updated Atto docs as per the feedback
WalkthroughThis update standardizes terminology across documentation by renaming "Data Generator" to "Test Data Generator" in navigation and content. Section headers and contextual links are clarified, and minor grammatical corrections are made. An informational note with troubleshooting resources is added to the Live Editor documentation. Changes
Suggested reviewers
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🔭 Outside diff range comments (2)
src/pages/docs/atto/ai-agents/test-data-generator.md (1)
2-8:page_idis now inconsistent with the new slugYou renamed everything to “test-data-generator” but kept the legacy
page_id: "data-generator".
This breaks deep-linking, anchor generation, and the new left-nav mapping.-page_id: "data-generator" +page_id: "test-data-generator"src/left-nav-title.json (1)
728-735: Duplicate keylive-editoroverrides previous entryJSON objects cannot have duplicate keys; the earlier mapping to
/docs/live-editor/is silently lost by the later one for Atto Copilot.
Rename one of them (example:copilot-live-editor) and adjust inbound links.-"live-editor": {"/docs/atto/generative-ai/copilot/live-editor/": "In Live Editor"}, +"copilot-live-editor": {"/docs/atto/generative-ai/copilot/live-editor/": "In Live Editor"},
🧹 Nitpick comments (5)
src/pages/docs/atto/generative-ai/copilot/live-editor.md (1)
61-63: Prefer relative paths for internal documentation linksHard-coding the full
https://testsigma.com/docs/...URL makes local previews, forked deployments, or future domain moves brittle. Use a root-relative link instead:-> For troubleshooting details and frequently asked questions, see [Terminal FAQs](https://testsigma.com/docs/live-editor/faqs-on-testsigma-terminal/). +> For troubleshooting details and frequently asked questions, see [Terminal FAQs](/docs/live-editor/faqs-on-testsigma-terminal/).src/pages/docs/atto/ai-agents/test-data-generator.md (2)
22-22: Minor grammar – missing definite article“This article discusses using Test Data Generator …”
Insert “the” for smoother reading.
-This article discusses using Test Data Generator to create +This article discusses using the Test Data Generator to create
49-50: Passive wording – tighten the sentence“Wait for the Test Data Generator to create data for the selected fields.”
Currently fine but consider “Generate” rather than “Wait” to keep instructions action-oriented.src/pages/docs/atto/generative-ai/generate-tests/mobile.md (1)
68-68: Verb duplication—remove trailing fragment
to generate multiple test cases based on the selected inputsis enough; “…are generated” is redundant.-Wait for **Generator Agent** to generate multiple test cases based on the selected inputs. +Wait for **Generator Agent** to generate multiple test cases based on the selected inputs.src/pages/docs/atto/ai-agents/generator.md (1)
64-65: Grammar: double verb in sentence“Wait for Generator Agent to generate multiple test cases are generated…”
Remove the trailing clause:
-Wait for Generator Agent to generate multiple test cases are generated based on the selected inputs. +Wait for the Generator Agent to generate multiple test cases based on the selected inputs.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/left-nav-title.json(1 hunks)src/pages/docs/atto/ai-agents/generator.md(3 hunks)src/pages/docs/atto/ai-agents/test-data-generator.md(3 hunks)src/pages/docs/atto/generative-ai/copilot/live-editor.md(1 hunks)src/pages/docs/atto/generative-ai/generate-tests/mobile.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
src/pages/docs/atto/ai-agents/test-data-generator.md
[uncategorized] ~22-~22: You might be missing the article “the” here.
Context: ...ata entry. This article discusses using Test Data Generator to create and customize ...
(AI_EN_LECTOR_MISSING_DETERMINER_THE)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: CI
🔇 Additional comments (4)
src/pages/docs/atto/generative-ai/generate-tests/mobile.md (1)
74-74: Heading demotion removed step number—align list numberingRemoving the number is fine, but update the preceding ordered list so that Markdown doesn’t restart numbering at 1 in rendered HTML. A simple
<!-- -->comment or explicit8.keeps the sequence intact.src/left-nav-title.json (1)
734-735: Newtest-data-generatorentry looks goodPath and title align with the renamed page.
src/pages/docs/atto/ai-agents/generator.md (2)
16-20: Contextual-link text updated—good catchThe new phrasing is clearer and matches the header anchor.
98-100: Clarified acceptance note—nice improvementThe conditional guidance is clearer and more actionable.
2 participants
Updated Atto docs as per the feedback
Summary by CodeRabbit