Skip to content

Conversation

wintonzheng
Copy link
Contributor

@wintonzheng wintonzheng commented Oct 2, 2025

Update API specifications by running fern api update.


📋 This PR updates the Skyvern OpenAPI specification with two key changes: extending the maximum session timeout from 240 to 1440 minutes (24 hours) and adding a new optional azure_folder_path field for Azure Blob storage configuration.

🔍 Detailed Analysis

Key Changes

  • Session Timeout Extension: Maximum timeout increased from 4 hours (240 minutes) to 24 hours (1440 minutes) while maintaining the 5-minute minimum
  • Azure Storage Enhancement: Added new optional azure_folder_path field to support folder-based organization within Azure Blob containers
  • API Documentation: Updated corresponding field descriptions to reflect the new timeout limits

Technical Implementation

flowchart TD
    A[Session Creation] --> B{Timeout Validation}
    B --> C[5 ≤ timeout ≤ 1440 minutes]
    C --> D[Session Started]
    
    E[Azure Storage Config] --> F[Container Name]
    E --> G[Folder Path - Optional]
    F --> H[Blob Storage Access]
    G --> H
Loading

Impact

  • Extended Session Duration: Users can now run longer automation sessions up to 24 hours, supporting complex workflows that require extended execution time
  • Improved Azure Integration: The new folder path parameter enables better organization and structure within Azure Blob storage containers
  • Backward Compatibility: Changes are non-breaking as the timeout extension only increases the maximum allowed value and the new field is optional

Created with Palmier


Important

Update API specifications in skyvern_openapi.json to increase timeout limit and add azure_folder_path field.

  • API Specification Update:
    • Increase maximum timeout from 240 to 1440 in skyvern_openapi.json.
    • Update description to reflect new timeout range (5 to 1440 minutes).
  • New Field Addition:
    • Add azure_folder_path field with string or null type in skyvern_openapi.json.

This description was created by Ellipsis for 21258dd. You can customize this summary. It will automatically update as commits are pushed.

Summary by CodeRabbit

  • New Features
    • Added support for specifying an Azure folder path in file uploads via an optional azure_folder_path field.
    • Increased the maximum browser session timeout to 1,440 minutes (24 hours), with validation now requiring a value between 5 and 1,440 minutes.

Copy link
Contributor

coderabbitai bot commented Oct 2, 2025

Walkthrough

Updated OpenAPI schema: increased CreateBrowserSessionRequest.timeout maximum from 240 to 1440 minutes with description adjusted, and added optional azure_folder_path field to FileUploadBlockYAML (string or null).

Changes

Cohort / File(s) Summary
OpenAPI schema updates
fern/openapi/skyvern_openapi.json
Raised CreateBrowserSessionRequest.timeout max to 1440 and updated description; added FileUploadBlockYAML.azure_folder_path (optional string or null).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • suchintan
  • jomido

Poem

A hop and a skip through schema fields new,
Timeout stretches to a day’s bright hue.
Azure paths whisper, “Upload here,”
Carrots aligned, the routes are clear.
With JSON maps and gentle cheer,
This bunny signs: ship it, dear! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title “Update API specifications with fern api update” is related to the changeset in that it indicates an API spec update, but it is overly generic and does not convey the specific enhancements—namely raising the CreateBrowserSessionRequest.timeout limit and introducing the azure_folder_path field—nor does it read clearly as a concise summary of the main changes. Please revise the title to succinctly reflect the key updates, for example “Increase CreateBrowserSessionRequest.timeout max to 1440 minutes and add azure_folder_path to FileUploadBlockYAML.”
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-openapi-spec-2025-10-02T00-28-38-257Z

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f43467c and 21258dd.

📒 Files selected for processing (1)
  • fern/openapi/skyvern_openapi.json (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Run tests and pre-commit hooks

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 21258dd in 1 minute and 7 seconds. Click for details.
  • Reviewed 40 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 3 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. fern/openapi/skyvern_openapi.json:4538
  • Draft comment:
    Updated timeout max from 240 to 1440. Confirm that sessions can run up to 24 hours as intended.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 0% <= threshold 50% The comment is asking the PR author to confirm their intention regarding the change in timeout value. This violates the rule against asking the author to confirm their intention or ensure behavior is intended.
2. fern/openapi/skyvern_openapi.json:4544
  • Draft comment:
    Timeout description updated to 1440 max. Ensure client docs and validations are aligned with this range.
  • Reason this comment was not posted:
    Comment looked like it was already resolved.
3. fern/openapi/skyvern_openapi.json:6323
  • Draft comment:
    New 'azure_folder_path' field added. Consider including a description or default value if applicable.
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None

Workflow ID: wflow_gkiyehLH56lfXesL

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

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