-
Notifications
You must be signed in to change notification settings - Fork 9
Filesearch using OpenAI assistants.md #393
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
base: main
Are you sure you want to change the base?
Conversation
* Filesearch Using OpenAI Assistants.md * Update Filesearch Using OpenAI Assistants.md
Warning Rate limit exceeded@akanshaaa19 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 0 minutes and 32 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests
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. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 2
🧹 Nitpick comments (10)
docs/4. Integrations/Filesearch Using OpenAI Assistants.md (10)
84-86
: Fix code formatting for node name and minor grammar.Backticks are mismatched around the node name; also minor grammar.
Apply this diff:
-- Provide a `Result Name` for the `Wait for Response node. -- In the example below, the result name is set as `question`. +- Provide a `Result Name` for the `Wait for Response` node. +- In the example below, the result name is set to `question`.
94-97
: Tighten wording and punctuation in webhook step.Remove double space, fix comma splice, keep UI terms consistent.
Apply this diff:
-- Select the `Function` from the dropdown. -- In the `Function` field, enter `filesearch-gpt`, this function name is pre-defined. -- Give the webhook `Result Name` - you can use any name. In the screenshot example, it’s named as `gptresponse`. +- Select `Function` from the dropdown. +- In the `Function` field, enter `filesearch-gpt`. This function name is pre-defined. +- Give the webhook `Result Name` — you can use any name. In the screenshot example, it’s named `gptresponse`.
87-89
: Don’t use emphasis as a heading (lint: MD036/MD049).Convert the emphasized line to plain text and make it a caption.
Apply this diff:
-_Screenshot of example flow set up is given below_ +Screenshot of the example flow setup:
102-107
: Reduce repetitive sentence starts; fix variable path typo (@result
→@results
).Also clarify parameter descriptions.
Apply this diff:
-#### Step 3: Click on Function Body (top right corner) and pass the following parameter +#### Step 3: Click on Function Body (top right corner) and pass the following parameters `{ "question": "@results.question", "assistant_id": "asst_xxxxx", "remove_citation":true }` -- In `question` parameter enter the flow variable containing the question asked by the user. In the given example `question` is the `result name`, hence provided `@result.question` in the question parameter. -- In `assistant_id` enter the assistant id obtained in step 4 of "How to Create an OpenAI Assistant in Glific" -- In `remove_citation` enter `true` to prevent cryptic citation marks from showing up in the response. +- `question`: the flow variable containing the user’s question. In this example, use `@results.question`. +- `assistant_id`: the Assistant ID obtained in Step 4 of "How to Create an OpenAI Assistant in Glific". +- `remove_citation`: set to `true` to prevent cryptic citation marks from showing up in the response.
130-131
: Hyphenate “follow-up” and tighten phrasing.Apply this diff:
-- In question parameter enter the flow variable containing the follow up question asked by the user. In the given example `result_5` is the result name, hence provided `@results.result_5` in the question parameter. +- In the `question` parameter, enter the flow variable containing the follow-up question. In this example, the result name is `result_5`, so use `@results.result_5`.
143-146
: Clarify UI label with code formatting.Apply this diff:
-- In the `Wait for Response` node, select has audio as the message response type. Also, give a result name. In the screenshot below, `audio_query` is used as the result name. +- In the `Wait for Response` node, select `Has audio` as the message response type. Also give a result name. In the screenshot below, `audio_query` is used as the result name.
149-153
: Fix grammar and consistency in the voice webhook step.Apply this diff:
-#### Step 2: Create Call a Webhook node +#### Step 2: Create a Call Webhook node - Select `Function` from the dropdown. -- In the `Function` field, enter `voice-filesearch-gpt` , this function name is pre-defined. -- Give the webhook result name - you can use any name. In the screenshot example, it’s named `gpt_voice`. +- In the `Function` field, enter `voice-filesearch-gpt`. This function name is pre-defined. +- Give the webhook `Result Name` — you can use any name. In the screenshot example, it’s named `gpt_voice`.
164-169
: Fix spelling (“source_language”), close the missing quote, and tighten phrasing.Apply this diff:
-- `assistant_id` is the assistant id obtained in step 4 of "How to Create an OpenAI Assistant in Glific. -- `source_langauge` is the expected language of the user. -- `target_language` is the language that the response voice note needs to be in. -- `remove_citation` pass as `true` to avoid citation marks to be part of the response voice note. +- `assistant_id` is the Assistant ID obtained in Step 4 of "How to Create an OpenAI Assistant in Glific". +- `source_language` is the expected language of the user. +- `target_language` is the language for the response voice note. +- `remove_citation`: pass `true` to avoid citation marks in the response.
172-175
: Fix backticks around node name and align example variable.Apply this diff:
-- Create a `Send Message node`. -- Use `@results.webhook_result-name.translated_text` to show the text response. +- Create a `Send Message` node. +- Use `@results.gpt_voice.translated_text` to show the text response.
62-69
: Standardize “Flow Editor” naming.You use “flow editor”, “Floweditor”, and “Flow Editor”. Recommend using “Flow Editor” consistently.
Apply this diff:
-This ID will be used in the webhook nodes in the flow editor. +This ID will be used in the webhook nodes in the Flow Editor. -## Using the OpenAI Assistant in Floweditor +## Using the OpenAI Assistant in Flow Editor
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/4. Integrations/Filesearch Using OpenAI Assistants.md
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/4. Integrations/Filesearch Using OpenAI Assistants.md
[style] ~106-~106: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ....questionin the question parameter. - In
assistant_id` enter the assistant id o...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
🪛 markdownlint-cli2 (0.17.2)
docs/4. Integrations/Filesearch Using OpenAI Assistants.md
18-18: Heading levels should only increment by one level at a time
Expected: h2; Actual: h3
(MD001, heading-increment)
18-18: Trailing punctuation in heading
Punctuation: ':'
(MD026, no-trailing-punctuation)
23-23: Trailing punctuation in heading
Punctuation: ':'
(MD026, no-trailing-punctuation)
32-32: Heading levels should only increment by one level at a time
Expected: h3; Actual: h4
(MD001, heading-increment)
79-79: Heading levels should only increment by one level at a time
Expected: h3; Actual: h4
(MD001, heading-increment)
87-87: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
87-87: Emphasis style
Expected: asterisk; Actual: underscore
(MD049, emphasis-style)
87-87: Emphasis style
Expected: asterisk; Actual: underscore
(MD049, emphasis-style)
92-92: Trailing punctuation in heading
Punctuation: '.'
(MD026, no-trailing-punctuation)
142-142: Heading levels should only increment by one level at a time
Expected: h3; Actual: h4
(MD001, heading-increment)
🪛 GitHub Actions: Test deployment
docs/4. Integrations/Filesearch Using OpenAI Assistants.md
[error] 160-160: MDX compilation failed: Could not parse expression with acorn (line 160, column 12). Step: docusaurus build.
🪛 GitHub Actions: Build and Deploy to Netlify
docs/4. Integrations/Filesearch Using OpenAI Assistants.md
[error] 160-160: MDX compilation failed: Could not parse expression with acorn at line 160, column 11.
🔇 Additional comments (2)
docs/4. Integrations/Filesearch Using OpenAI Assistants.md (2)
193-195
: Verify pricing/credit claims and dates.The section asserts OpenAI support and $100 credits until August 2026. Please verify these externally (website, legal/partner comms) before publishing.
Would you like me to draft a short “Pricing last verified on ” note and/or place this behind a versioned docs flag?
12-27
: Nice restructure and clarity.The intro, use-cases, and overview are crisp and make the user journey clear.
|
||
<img width="1409" alt="Screenshot 2024-11-18 at 2 54 10 PM" src="https://github.com/user-attachments/assets/1aa8bcaa-b718-4c95-bca9-5b34059b077e" /> | ||
### Use this when: |
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.
🛠️ Refactor suggestion
Normalize heading levels and remove trailing punctuation (lint: MD001, MD026).
Headings jump levels and include trailing punctuation. Align with H1 -> H2 -> H3 progression and remove colons/periods.
Apply this diff:
-### Use this when:
+## Use this when
-### This document guides you through three main parts:
+## This document covers three parts
-#### Step 1: Create a new AI Assistant
+### Step 1: Create a new AI Assistant
-#### Step 1: Get User Question
+### Step 1: Get user question
-#### Step 2: Add a Call Webhook node. This is where we integrate the OpenAI Assistant.
+### Step 2: Add a Call Webhook node
+This is where we integrate the OpenAI Assistant.
-#### Step 1: Capture end user’s voice input
+### Step 1: Capture end user’s voice input
Also applies to: 23-23, 32-32, 79-79, 92-92, 142-142
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
18-18: Heading levels should only increment by one level at a time
Expected: h2; Actual: h3
(MD001, heading-increment)
18-18: Trailing punctuation in heading
Punctuation: ':'
(MD026, no-trailing-punctuation)
🤖 Prompt for AI Agents
In docs/4. Integrations/Filesearch Using OpenAI Assistants.md around line 18
(also applies to lines 23, 32, 79, 92, and 142), several headings jump levels
and include trailing punctuation; normalize heading hierarchy so they follow H1
-> H2 -> H3 progression and remove trailing colons/periods from the heading text
(e.g., change an H3 that improperly follows an H1 to the correct H2/H3 level and
strip any trailing ":" or "."), and update the affected heading lines
accordingly to satisfy MD001 and MD026.
🚀 Deployed on https://deploy-preview-393--glific-docs.netlify.app |
Summary by CodeRabbit