-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Docs - troubleshooting #15762
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
Docs - troubleshooting #15762
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Warning Rate limit exceeded@andrewjschuang has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 22 minutes and 41 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)
WalkthroughThe pull request updates the troubleshooting documentation by reorganizing content in the main Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant D as Documentation
participant API as API Gateway
participant WR as Workflow Runner
U->>D: Reads "How do I invoke another workflow?" section
U->>API: Sends HTTP request to trigger workflow
API->>WR: Passes trigger request
WR-->>API: Returns execution result
API-->>U: Provides workflow response
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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 (
|
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: 0
🧹 Nitpick comments (10)
docs-v2/pages/troubleshooting.mdx (10)
11-21: New "How do I share my workflow with Support?" Section Clarity
This newly added section provides clear, step‐by‐step guidance for users to share their project/workflow settings with support. The inclusion of the expected URL format (as a code block) is a nice touch that illustrates what users should see in their browser.
Consider adding hyperlinks directly to "Project Settings" and "Workflow Settings" if applicable, to further enhance usability.
35-37: "I need help with my GitHub Synced Project" Section – Link Verification
This new section succinctly directs users to the FAQ. Since the FAQ file was removed and its content integrated into the main document, please verify that the anchor[See FAQ here](/workflows/git/#faq)correctly points to the updated FAQ content.
51-56: Enhancing the "How do I invoke another workflow?" Guidance
The section clearly presents two alternatives for invoking another workflow—using the Trigger Workflow action and making an HTTP request. To further assist users, consider including a brief code snippet or an example demonstrating the HTTP request method.
63-63: Subheading Update: "Webhook-based instant sources"
The updated subheading precisely defines the type of source. This clear designation improves readability.
73-73: Subheading Update: "Timer-based polling sources"
The heading now aligns with the overall documentation style. A minor suggestion: review the accompanying explanatory text (around lines 74–79) for potential wordiness (per static analysis hints) to ensure conciseness.
107-113: Revisiting the "Undeployed changes" Warning
The warning content is informative, but it could be more concise and direct. For instance, consider shortening the heading and the following explanation. A possible diff might be:-### Undeployed changes — You have made changes to this workflow. Deploy the latest version from the editor +### Undeployed ChangesThen, follow with a briefer note on the importance of deploying changes.
🧰 Tools
🪛 LanguageTool
[style] ~107-~107: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~111-~111: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~113-~113: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...(MAKE_CHANGES)
198-205: "Out of date actions or sources" Section Clarity
This section is clear and concise, effectively informing users about the need to update actions or re-create sources when new versions are released. Double-check that all interactive elements (links and update prompts) function as intended.🧰 Tools
🪛 LanguageTool
[grammar] ~202-~202: This phrase is duplicated. You should probably use “in the” only once.
Context: ...actions-to-the-latest-version) is shown in the in the top right of the action if the componen...(PHRASE_REPETITION)
218-224: Enhanced Block for NPM Package Warnings
The updated callout provides a useful warning regarding the use of NPM packages in components. For additional clarity, consider including a link or reference for users who need guidance on managing package downgrades.🧰 Tools
🪛 LanguageTool
[style] ~222-~222: Consider a shorter alternative to avoid wordiness.
Context: ...1c/components/slack/slack.app.mjs#L1). In order to downgrade these packages, you'll need t...(IN_ORDER_TO_PREMIUM)
248-253: Data Storage and Retrieval Guidance
The section concisely explains how to store and retrieve data across workflow executions, offering both an external and a built-in solution. You might consider adding an example or a link to a practical guide for users unfamiliar with database integration on Pipedream.
258-269: Optimizing Workflow Speed
The "How can my workflow run faster?" section offers actionable recommendations (e.g., increasing memory, returning static HTTP responses, simplifying workflows, and activating warm workers). The bullet-point list is well organized; consider adding concrete examples to further illustrate each tip.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs-v2/pages/troubleshooting.mdx(10 hunks)docs-v2/pages/troubleshooting/_meta.tsx(0 hunks)docs-v2/pages/troubleshooting/faq.mdx(0 hunks)
💤 Files with no reviewable changes (2)
- docs-v2/pages/troubleshooting/_meta.tsx
- docs-v2/pages/troubleshooting/faq.mdx
🧰 Additional context used
🪛 LanguageTool
docs-v2/pages/troubleshooting.mdx
[style] ~79-~79: Consider a shorter alternative to avoid wordiness.
Context: ...aylist) require we poll their endpoints in order to fetch new events. ### Where do I find ...
(IN_ORDER_TO_PREMIUM)
[style] ~107-~107: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~111-~111: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~113-~113: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...
(MAKE_CHANGES)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: Lint Code Base
- GitHub Check: validate-links
🔇 Additional comments (6)
docs-v2/pages/troubleshooting.mdx (6)
127-197: Restructured Error Handling Sections
The error sections—from Runtime Quota Exceeded to Pipedream Internal Error—are now segmented into distinct, clearly labeled subsections. This reorganization makes it easier for users to locate and understand troubleshooting steps for specific issues.
Please verify that dynamic values such as{process.env.FUNCTION_PAYLOAD_LIMIT}and{process.env.MAX_WORKFLOW_QUEUE_SIZE}render properly at runtime.🧰 Tools
🪛 LanguageTool
[uncategorized] ~137-~137: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...**. Often, you're making a request to a third party API that doesn't respond in the time yo...(EN_COMPOUND_ADJECTIVE_INTERNAL)
[typographical] ~147-~147: Consider adding a comma after ‘Typically’ for more clarity.
Context: ...flow. That incurs some memory overhead. Typically this happens when you have more than 8-...(RB_LY_COMMA)
[typographical] ~147-~147: If specifying a range, consider using an en dash instead of a hyphen.
Context: ...ly this happens when you have more than 8-10 steps. When you see an OOM error on a w...(HYPHEN_TO_EN)
244-247: "Replay Workflow Events" Section Clarity
This brief section clearly explains that workflow event replay isn’t supported via the API but can be done manually through Event History. It adds useful context for troubleshooting.
254-257: Delaying Workflow Execution Made Simple
This section straightforwardly instructs users to use the built-in Delay actions to pause a workflow. The guidance is clear and needs no further changes.
270-275: Saving Common Functions as Steps
This section provides concise advice on creating custom components via the Component API or publishing to public registries. The content is practical and clear.
276-279: Puppeteer Support Confirmation
The section confirming Puppeteer support is brief and effectively directs users to the detailed docs. It’s clear and concise.
280-282: Playwright Support Confirmation
Mirroring the Puppeteer section, this part clearly confirms Playwright support and provides direct access to further details. The consistency between the two sections is commendable.
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: 0
🧹 Nitpick comments (4)
docs-v2/pages/troubleshooting.mdx (4)
57-61: Renamed Workflow Invocation Section
The heading is now changed to "How do I invoke another workflow?" and the section explains the available approaches—using the Trigger Workflow action and HTTP webhook calls.
Please verify that the old section ("My workflow isn't working" at line 47) is either removed or consolidated to avoid any duplication.
69-69: Improve Clause Clarity in Trigger Logging Paragraph
On this line, consider inserting a comma before "and" to separate the independent clauses for improved readability. For example:-... it may fail to log the execution and you won't see any new events and logs. +... it may fail to log the execution, and you won't see any new events and logs.🧰 Tools
🪛 LanguageTool
[uncategorized] ~69-~69: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...t once, it may fail to log the execution and you won't see any new events and logs. ...(COMMA_COMPOUND_SENTENCE)
123-129: Undeployed Changes Section Wording
The undeployed changes section communicates that users should deploy their latest changes. To further enhance clarity and brevity, consider rephrasing the header. For example:-### Undeployed changes — You have made changes to this workflow. Deploy the latest version from the editor +### Undeployed Changes – Deploy Your Latest Workflow Version🧰 Tools
🪛 LanguageTool
[style] ~123-~123: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~127-~127: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~129-~129: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...(MAKE_CHANGES)
242-258: Packages Consuming Storage
The section details how importing entire packages (e.g., lodash) can lead to excessive storage use and shows improved import practices with clear code examples. Ensure that the code snippet formatting remains consistent with the rest of the document.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs-v2/pages/troubleshooting.mdx(8 hunks)docs-v2/pages/workflows/building-workflows/settings/index.mdx(1 hunks)docs-v2/pages/workflows/building-workflows/triggers.mdx(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- docs-v2/pages/workflows/building-workflows/triggers.mdx
- docs-v2/pages/workflows/building-workflows/settings/index.mdx
🧰 Additional context used
🪛 LanguageTool
docs-v2/pages/troubleshooting.mdx
[uncategorized] ~69-~69: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...t once, it may fail to log the execution and you won't see any new events and logs. ...
(COMMA_COMPOUND_SENTENCE)
[style] ~87-~87: Consider a shorter alternative to avoid wordiness.
Context: ...aylist) require we poll their endpoints in order to fetch new events. ### Where do I find ...
(IN_ORDER_TO_PREMIUM)
[style] ~123-~123: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~127-~127: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~129-~129: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...
(MAKE_CHANGES)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: validate-links
- GitHub Check: Lint Code Base
🔇 Additional comments (29)
docs-v2/pages/troubleshooting.mdx (29)
19-22: New Support Contact Section Added
The new "How do I contact Pipedream Support?" section provides clear guidance for users to fill out a support request form. Keeping support information prominent is essential for timely assistance.
23-27: New Workflow Sharing Section
The newly introduced "How do I share my workflow with Support?" section offers concise instructions on sharing project settings with support. This additional guidance enhances the user experience for those needing help beyond a support ticket.
29-33: Example URL for Workflow Sharing
Presenting a formatted URL example reinforces the expected structure when sharing a workflow. This practical example will help reduce user error when submitting support requests.
89-98: Event Source ID Section
The "Where do I find my event source's ID?" section adds clarity by distinguishing it from workflow IDs. The inclusion of an example URL helps users quickly identify the correct ID format.
99-105: Trigger Logs Section
Clear instructions on finding the trigger logs are provided, along with a helpful URL example. This is an important troubleshooting detail that should make it easier for users to diagnose trigger issues.
107-113: Paused Trigger Explanation
The explanation regarding why a trigger might be paused is straightforward and guides users to check logs and re-enable the source if needed.
117-121: Asynchronous Execution Warning
The "Code was still running when the step ended" warning is clear and supported by a reference to Node.js documentation, which bolsters its usefulness for troubleshooting asynchronous code issues.
133-137: Limit Exceeded Errors
This section explains the various limits (runtime, memory, etc.) clearly and provides a useful link for more details. The information is accurate and well organized.
137-142: Quota Exceeded Error Explanation
The description for the "Quota Exceeded" error is clear, outlining the daily credit constraints on the Free tier and offering an upgrade path. A brief note suggesting users reach out for further support could be beneficial but is optional.
143-145: Runtime Quota Exceeded Details
This section explains the runtime quota limits effectively. Please verify that the dynamic content placeholder{process.env.DAILY_TESTING_LIMIT}is rendered properly in the final documentation.
145-153: Timeout Error Guidance
The guidance for addressing timeout errors is detailed and actionable. The step-by-step advice, including checking code performance and adjusting timeout settings, provides valuable troubleshooting directions.🧰 Tools
🪛 LanguageTool
[uncategorized] ~153-~153: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...**. Often, you're making a request to a third party API that doesn't respond in the time yo...(EN_COMPOUND_ADJECTIVE_INTERNAL)
156-164: Out of Memory Error Section
The explanation for out of memory errors is comprehensive, outlining the primary causes and providing practical remediation advice. The recommendations are well-articulated.🧰 Tools
🪛 LanguageTool
[typographical] ~163-~163: Consider adding a comma after ‘Typically’ for more clarity.
Context: ...flow. That incurs some memory overhead. Typically this happens when you have more than 8-...(RB_LY_COMMA)
[typographical] ~163-~163: If specifying a range, consider using an en dash instead of a hyphen.
Context: ...ly this happens when you have more than 8-10 steps. When you see an OOM error on a w...(HYPHEN_TO_EN)
165-168: Rate Limit Exceeded Information
This section clearly describes the rate limiting behavior for HTTP interfaces, which should help users understand why they might encounter such errors.
171-175: Request Entity Too Large Details
The provided workarounds for handling large HTTP payloads are clear and include appropriate links for further reading. This added detail is very useful.
180-184: Function Payload Limit Exceeded
This explanation outlines the constraints on function payload sizes effectively. Please ensure that the placeholder{process.env.FUNCTION_PAYLOAD_LIMIT}is replaced or rendered correctly when viewed by end users.
188-193: JSON Nested Property Limit Exceeded
The concise description of the JSON nested property limit and its common causes is clear. Adding a brief troubleshooting tip here may further assist users encountering this error, though the current explanation is sufficient.
194-198: Event Queue Full Error
This section clearly explains what happens when the event queue is full and provides a link for increasing queue size. The information is straightforward and should help users address the issue.
200-206: Credit Budget Exceeded Notification
Clear instructions are provided for managing credit budget limits, along with a direct link to billing settings for adjustments.
208-213: Pipedream Internal Error Overview
The internal error section gives a solid overview of potential internal issues and their troubleshooting paths. Consider adding a reference to additional support resources if applicable.
214-221: Out of Date Actions or Sources
The description explains how out-of-date components are handled. The content is clear and informs users about the update process for actions and sources.🧰 Tools
🪛 LanguageTool
[grammar] ~218-~218: This phrase is duplicated. You should probably use “in the” only once.
Context: ...actions-to-the-latest-version) is shown in the in the top right of the action if the componen...(PHRASE_REPETITION)
222-230: New Package Versions Issues
This section effectively outlines the potential problems with new package versions and suggests downgrading as a remedy. Including a brief note on version pinning best practices is a nice touch.🧰 Tools
🪛 LanguageTool
[uncategorized] ~224-~224: Did you mean “its” (the possessive pronoun)?
Context: ... the building of the workflow or during it's execution, it may cause a `Pipedream In...(ITS_PREMIUM)
[style] ~230-~230: Consider using a different verb for a more formal wording.
Context: ...m Internal Error`. You can potentially fix this issue by downgrading packages by p...(FIX_RESOLVE)
234-240: Callout Warning on NPM Package Import
The callout is well-crafted, warning users about importing entire NPM packages and providing an alternative with specific module imports. This should help avoid unnecessary overhead in workflows.🧰 Tools
🪛 LanguageTool
[style] ~238-~238: Consider a shorter alternative to avoid wordiness.
Context: ...1c/components/slack/slack.app.mjs#L1). In order to downgrade these packages, you'll need t...(IN_ORDER_TO_PREMIUM)
260-263: Replaying Workflow Events Information
This brief section clarifies that workflow events cannot be replayed via the API but that users can bulk select and replay failed events manually. The guidance is concise and sets the correct expectations.
264-269: Data Storage Across Executions
The instructions for storing and retrieving data—whether using an external database or the built-in key-value store—are practical. A slight enhancement might be to mention when one approach may be favored over the other, but the current content is sufficiently clear.
270-273: Delaying Workflow Execution
The guidance on using built-in Delay actions is brief and to the point, offering users a quick method to delay workflow execution as needed.
274-283: Improving Workflow Speed
The section provides a comprehensive list of strategies to speed up workflows, including increasing memory, returning static HTTP responses, simplifying workflows, and activating warm workers. The detailed recommendations are both actionable and informative.
286-290: Saving Common Functions as Steps
This section explains how to create custom triggers and actions as well as publish common functions. Including this information supports best practices for code reuse and component development.
292-295: Puppeteer Support Information
A succinct confirmation of Puppeteer support is provided along with a link to detailed documentation. The content is clear and helpful for users needing browser automation guidance.
296-299: Playwright Support Confirmation
Mirroring the Puppeteer support note, this section confirms Playwright support and directs users to the appropriate documentation. The consistency between these sections is excellent.
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: 0
🧹 Nitpick comments (4)
docs-v2/pages/troubleshooting.mdx (4)
61-68: Trigger Troubleshooting Instructions
The troubleshooting guidance for triggers is well structured. For improved clarity, consider adding a comma before “and” in the sentence on line 67 to better separate the independent clauses.🧰 Tools
🪛 LanguageTool
[uncategorized] ~67-~67: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...t once, it may fail to log the execution and you won't see any new events and logs. ...(COMMA_COMPOUND_SENTENCE)
69-77: Webhook-based Instant Sources Explanation
The explanation for webhook-based instant sources is informative and appropriately detailed. A slight rewording for conciseness (as suggested by static analysis in similar contexts) might further improve readability, although the content is already quite clear.🧰 Tools
🪛 LanguageTool
[duplication] ~71-~71: Possible typo: you repeated a word.
Context: ...ed immediately. But because events come in in real-time, most will not automatica...(ENGLISH_WORD_REPEAT_RULE)
79-85: Timer-based Polling Sources Description
The section describing timer-based polling sources provides valuable details (including an image) and explains how polling works for historical event retrieval. Consider simplifying the phrasing around “polling their endpoints” to make it even more concise.🧰 Tools
🪛 LanguageTool
[style] ~85-~85: Consider a shorter alternative to avoid wordiness.
Context: ...aylist) require we poll their endpoints in order to fetch new events. ### Where do I find ...(IN_ORDER_TO_PREMIUM)
121-127: Undeployed Changes Notification – Consider Rephrasing
The section notifying users about "Undeployed changes" is important for ensuring deployments are up to date. However, the current wording (lines 121–127) is a bit lengthy. Consider rephrasing or shortening the text to improve clarity and impact.🧰 Tools
🪛 LanguageTool
[style] ~121-~121: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~125-~125: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...(MAKE_CHANGES)
[style] ~127-~127: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...(MAKE_CHANGES)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs-v2/pages/troubleshooting.mdx(8 hunks)
🧰 Additional context used
🪛 LanguageTool
docs-v2/pages/troubleshooting.mdx
[uncategorized] ~67-~67: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...t once, it may fail to log the execution and you won't see any new events and logs. ...
(COMMA_COMPOUND_SENTENCE)
[style] ~85-~85: Consider a shorter alternative to avoid wordiness.
Context: ...aylist) require we poll their endpoints in order to fetch new events. ### Where do I find ...
(IN_ORDER_TO_PREMIUM)
[style] ~121-~121: Consider shortening or rephrasing this to strengthen your wording.
Context: ...ils. ### Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~125-~125: Consider shortening or rephrasing this to strengthen your wording.
Context: ...w: > Undeployed changes — You have made changes to this workflow. Deploy the latest versio...
(MAKE_CHANGES)
[style] ~127-~127: Consider rephrasing this to strengthen your wording.
Context: ...from the editor This means that you've made some changes to your workflow that you haven't yet d...
(MAKE_CHANGES)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: Lint Code Base
- GitHub Check: validate-links
🔇 Additional comments (21)
docs-v2/pages/troubleshooting.mdx (21)
19-22: New Support Contact Section Added
The new section clearly instructs users on how to contact Pipedream Support via the provided request form link. The content is concise and fits well within the troubleshooting context.
23-31: Workflow Sharing Instructions Added
The added subsection "How do I share my workflow with Support?" offers step‐by‐step guidance—including an example URL—to help users share project settings with support. Ensure the URL format and instructions remain in sync with any future UI updates.
35-44: Consistency Check for ID Sections
The document now includes a section on "Where do I find my workflow's ID?" (lines 35–44) alongside a later section on "Where do I find my event source's ID?" (lines 87–96). Please verify that these two sections are intentionally distinct and consider whether consolidating or clarifying their differences might reduce user confusion.
45-53: Clarification Needed for Troubleshooting Sections
The "My workflow isn't working" section (lines 45–53) covers basic troubleshooting steps. However, immediately following it is the "How do I invoke another workflow?" section (lines 55–60). It would help to clarify the distinct objectives of these sections so that users understand which guidance applies to troubleshooting execution issues versus invoking workflows.🧰 Tools
🪛 LanguageTool
[style] ~47-~47: Consider using a different verb for a more formal wording.
Context: ...ollowing steps, checking to see if they fix the problem: 1. Make a trivial change ...(FIX_RESOLVE)
55-60: Workflow Invocation Guidance
The newly added "How do I invoke another workflow?" section clearly explains the use of the Trigger Workflow action and HTTP requests to invoke workflows. The instructions are succinct and actionable.
87-96: Event Source ID Guidance
The new section "Where do I find my event source's ID?" is clearly presented with a practical example. The instructions and code block example effectively show users how to locate the source ID.
97-103: Accessing Trigger Logs
The instructions for finding trigger logs (lines 97–103) are straightforward and include an example URL. This clear guidance should help streamline troubleshooting for users experiencing trigger issues.
105-109: Explanation for Paused Triggers
The "Why is my trigger paused?" section provides a succinct explanation along with actionable advice for resolving the issue. The content is both clear and helpful.
115-120: Asynchronous Issue Warning
The "Code was still running when the step ended" warning clearly describes the pitfalls of unfinished asynchronous operations, with a useful link to the Node.js documentation. This practical advice is well communicated.
129-137: Error Handling Introduction
The beginning of the Errors section—which covers "Limit Exceeded Errors" and "Quota Exceeded" (lines 129–137)—is well organized. The error categories are clearly delineated and provide users with immediate context for troubleshooting.
141-230: Comprehensive Error Categories Presented
From "Runtime Quota Exceeded" through "Pipedream Internal Error" (lines 141–230), the document now includes a robust, detailed listing of potential runtime and configuration errors. Each error category is clearly labeled and accompanied by additional guidance and relevant links. This depth of detail will be very helpful for advanced troubleshooting.🧰 Tools
🪛 LanguageTool
[uncategorized] ~151-~151: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...**. Often, you're making a request to a third party API that doesn't respond in the time yo...(EN_COMPOUND_ADJECTIVE_INTERNAL)
[typographical] ~161-~161: Consider adding a comma after ‘Typically’ for more clarity.
Context: ...flow. That incurs some memory overhead. Typically this happens when you have more than 8-...(RB_LY_COMMA)
[typographical] ~161-~161: If specifying a range, consider using an en dash instead of a hyphen.
Context: ...ly this happens when you have more than 8-10 steps. When you see an OOM error on a w...(HYPHEN_TO_EN)
[grammar] ~216-~216: This phrase is duplicated. You should probably use “in the” only once.
Context: ...actions-to-the-latest-version) is shown in the in the top right of the action if the componen...(PHRASE_REPETITION)
[uncategorized] ~222-~222: Did you mean “its” (the possessive pronoun)?
Context: ... the building of the workflow or during it's execution, it may cause a `Pipedream In...(ITS_PREMIUM)
[style] ~228-~228: Consider using a different verb for a more formal wording.
Context: ...m Internal Error`. You can potentially fix this issue by downgrading packages by p...(FIX_RESOLVE)
232-236: Callout for Package Downgrading
The callout section regarding the downgrading of NPM or PyPI packages (lines 232–236) provides useful guidance for users encountering package-related issues. Just ensure that the referenced links and repository paths remain up-to-date over time.🧰 Tools
🪛 LanguageTool
[style] ~236-~236: Consider a shorter alternative to avoid wordiness.
Context: ...1c/components/slack/slack.app.mjs#L1). In order to downgrade these packages, you'll need t...(IN_ORDER_TO_PREMIUM)
238-242: Storage Consumption Warning
The section on "Packages consuming all available storage" concisely explains a potential pitfall with package imports. This is a valuable pointer for users facing internal errors due to resource limitations.
244-256: Lodash Import Guidance
The provided code examples clearly illustrate the difference between importing the entire lodash package versus just the needed module. This is an effective, practical tip for optimizing package size.
258-261: Replay Workflow Events Guidance
The answer to whether workflow events can be replayed is succinct and directs users to the relevant Event History feature. This information is clear and useful.
262-267: Data Storage Options Clarified
The section on storing and retrieving data across workflow executions explains both custom data store options and the built-in key-value store. The guidance is actionable and clear.
268-271: Workflow Delay Instructions
The instructions for delaying workflow execution are brief and direct, effectively guiding users to use the built-in Delay actions.
272-283: Performance Optimization Tips
The "How can my workflow run faster?" section offers practical, bullet-pointed tips to improve performance. The suggestions are clear and well-organized.
284-289: Saving Common Functions Guidance
The guidance on saving common functions as steps is clear and supports best practices in reusability. The inclusion of links to additional resources further supports user implementation.
290-293: Puppeteer Support Confirmation
The Puppeteer support section confirms compatibility and directs users to more detailed documentation, which is very helpful.
294-297: Playwright Support Confirmation
The section confirming Playwright support is concise and properly directs users to the corresponding documentation for further details.
WHY
Summary by CodeRabbit