-
Notifications
You must be signed in to change notification settings - Fork 180
Add docs for enhance prompt #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
242dffb to
d96b30e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 242dffb in 58 seconds
More details
- Looked at
65lines of code in2files - Skipped
0files when reviewing. - Skipped posting
4drafted comments based on config settings.
1. docs/advanced-usage/enhance-prompt.md:46
- Draft comment:
Consider adding a newline at the end of the file for consistency with POSIX standards and to satisfy Prettier formatting. - Reason this comment was not posted:
Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 0% vs. threshold = 50%
This is a new file being added, and the comment is about a formatting issue. According to the rules, we should rely on automated tools like Prettier to enforce formatting standards. The comment suggests this is something Prettier would catch anyway.
The comment is technically correct about POSIX standards, but is this something that needs manual review?
Since Prettier is mentioned in the rules as the tool that should automatically enforce formatting, this kind of comment should be handled by automation, not manual review.
The comment should be deleted since formatting issues like trailing newlines should be handled automatically by Prettier, which is mentioned in the rules as the designated tool for this.
2. sidebars.ts:39
- Draft comment:
Ensure the new 'advanced-usage/enhance-prompt' sidebar item is in the desired logical order; otherwise, it looks fine. - Reason this comment was not posted:
Confidence changes required:50%<= threshold50%
None
3. docs/advanced-usage/enhance-prompt.md:46
- Draft comment:
Please add a newline at the end of the file to comply with standard style guidelines. - Reason this comment was not posted:
Marked as duplicate.
4. sidebars.ts:39
- Draft comment:
The new 'advanced-usage/enhance-prompt' entry is correctly added. Ensure this ordering matches the intended documentation structure. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%
None
Workflow ID: wflow_ArPwUtfNfBdyCLdx
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested. Incremental review on d96b30e in 57 seconds
More details
- Looked at
65lines of code in2files - Skipped
0files when reviewing. - Skipped posting
3drafted comments based on config settings.
1. sidebars.ts:39
- Draft comment:
New sidebar entry for 'advanced-usage/enhance-prompt' added. Ensure consistency with existing documentation structure. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%
None
2. docs/advanced-usage/enhance-prompt.md:46
- Draft comment:
Missing newline at end of file. Please add a newline to comply with standard file formatting. - Reason this comment was not posted:
Marked as duplicate.
3. sidebars.ts:39
- Draft comment:
The addition of 'advanced-usage/enhance-prompt' in the sidebar is correctly placed. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%
None
Workflow ID: wflow_Ds7fghVgfDfRnAcg
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
d96b30e to
23f7cd3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on 23f7cd3 in 36 seconds
More details
- Looked at
64lines of code in2files - Skipped
0files when reviewing. - Skipped posting
5drafted comments based on config settings.
1. docs/advanced-usage/enhance-prompt.md:1
- Draft comment:
Docs look clear and comprehensive. Consider adding a brief screenshot or visual aid to enhance clarity for UI-related instructions. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%
None
2. sidebars.ts:39
- Draft comment:
Sidebar update correctly includes the new 'enhance-prompt' doc. Ensure ordering remains consistent with other advanced usage topics. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%
None
3. docs/advanced-usage/enhance-prompt.md:11
- Draft comment:
Consider using 'Roo Code' instead of 'Roo' for consistency with other references. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%
None
4. docs/advanced-usage/enhance-prompt.md:25
- Draft comment:
Consider rephrasing 'listed out support prompts' to 'a list of supported prompts' for improved clarity. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%
None
5. sidebars.ts:39
- Draft comment:
New entry 'advanced-usage/enhance-prompt' added. Ensure this ordering aligns with your intended documentation structure. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%
None
Workflow ID: wflow_9dVStS9mP6QPa5Fy
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
Important
Adds documentation for the "Enhance Prompt" feature and updates the sidebar to include it.
enhance-prompt.mdunderdocs/advanced-usage/to document the "Enhance Prompt" feature.sidebars.tsto includeadvanced-usage/enhance-promptin the "Advanced Usage" category.This description was created by
for 23f7cd3. It will automatically update as commits are pushed.