GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — feat(integrations): add Snowflake PAT integration by BillLeoutsakosvl346 · Pull Request #6407 · simstudioai/sim

Conversation

@BillLeoutsakosvl346

@BillLeoutsakosvl346 BillLeoutsakosvl346 commented Aug 8, 2026 •
edited by waleedlatif1
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Summary

  • add a PAT-authenticated Snowflake block with 21 SQL, structured data, warehouse, task, schema, and procedure operations
  • centralize bounded Snowflake SQL API transport, identifier and binding safety, partition handling, and DML statistics
  • add transport and SQL-builder tests, generated metadata/catalog/docs, templates, and integration skills

Type of Change

  • New feature

Testing

43 tests / 2 files. Repo lint, type-check, and all audit scripts pass (tool-metadata:check, integration-catalog:check, mship-tools:check, check:bare-icons, check:icon-paths, check:tool-request-boundary, check:api-validation).

Validated against the live Snowflake SQL API v2 and SQL reference docs. Fixes from that pass:

SQL generation

  • PARSE_JSON is lifted out of the VALUES clause into a projecting SELECT; the previous form is rejected by Snowflake for any object or array value
  • string literals escape backslashes as well as quotes, closing a COPY option injection through the user-or-llm stagePath and pattern fields
  • stage paths reject --, which previously commented out every following clause
  • COPY INTO clauses are emitted in the documented positional order
  • introspect_schema excludes only view types, so temporary, external, and event tables are visible
  • MERGE uses plain equality and rejects null or duplicate match keys
  • rows and bound-value bytes are bounded for every statement, measured in UTF-8
  • qualified task names are rejected, which TASK_HISTORY silently ignores

Transport

  • DML stats read from the documented top-level ResultSet property
  • Link-header and 391908 paging removed (they belong to the retired API); partition completeness now reports unknown rather than falsely complete
  • a 2xx status is required before trusting a success SQLSTATE
  • response bodies are capped and invalid session context names fail closed

Conventions

  • shared params are inlined per tool instead of cross-file spreads, matching every other integration and letting the docs generator emit host and apiKey

Not yet exercised against a live Snowflake account. Worth one smoke run before relying on: ? binds inside FROM (VALUES ...) (insert/upsert), TO_TIMESTAMP_LTZ(?) in TASK_HISTORY time-range args (BCR-1410 lists only RESULT_LIMIT and TASK_NAME as bind-accepting, so startTime/endTime may be silently ignored), and ->> over /api/v2/statements (get_warehouse).

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with eyes emoji
All reactions
  • 👀 1 reaction
@gitguardian

gitguardian Bot commented Aug 8, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel

vercel Bot commented Aug 8, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 8, 2026 11:28pm

Request Review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cursor

cursor Bot commented Aug 8, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

PR Summary

Medium Risk
New integration executes parameterized SQL and DML against customer Snowflake accounts using PAT credentials; mistakes or injection bugs could affect production data, though the change is additive and the PR describes extensive SQL/transport hardening and tests.

Overview
Adds a Snowflake integration authenticated with a programmatic access token (PAT), registered in the block registry, integration catalog, and docs navigation.

The new block exposes 21 operations across SQL execution (including async runs, statement polling/cancel, and partitioned results), structured row insert/update/upsert/delete, stage-based COPY loads, warehouse list/get/resume/suspend, task definition and run history, schema introspection, and stored procedure calls. Operation-specific sub-blocks cover bindings, match columns, COPY options, task history filters, and shared execution settings (role, warehouse, timeouts, row limits).

Docs include a generated snowflake.mdx reference for every action’s inputs/outputs. SnowflakeIcon is updated to Snowflake’s branded SVG in both docs and app icon sets, with snowflake mapped in icon registries.

Reviewed by Cursor Bugbot for commit 36b3aa3. Configure here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the feat/snowflake-integration branch from 745eccd to 910c5b5 Compare August 8, 2026 05:25
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

GitGuardian finding 35885962 was a false positive on the built-in JavaScript URL credential property, not a hardcoded secret. The host validator now uses an equivalent canonical-URL comparison, and the single commit was rewritten so the flagged token is absent from PR history.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@greptile-apps

greptile-apps Bot commented Aug 8, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a PAT-authenticated Snowflake integration with SQL execution, structured data operations, warehouse and task management, schema introspection, and stored-procedure support.

  • Adds 21 Snowflake tools and a workflow block with generated registry, metadata, output, catalog, and documentation updates.
  • Centralizes Snowflake SQL API response transformation, bounded payload handling, identifier validation, bindings, partition metadata, and DML statistics.
  • Adds focused SQL-builder and transport tests, including asynchronous HTTP 202 statement-handle preservation.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/tools/snowflake/utils.ts Implements shared Snowflake request validation and response transformation; the reviewed pending-response path correctly preserves HTTP 202 statement handles.
apps/sim/tools/snowflake/sql.ts Centralizes bounded and validated SQL generation for structured data, warehouse, task, schema, and procedure operations.
apps/sim/blocks/blocks/snowflake.ts Defines the Snowflake workflow block and maps its operation-specific inputs to the registered tools.
apps/sim/tools/snowflake/utils.test.ts Covers transport limits, response handling, partition metadata, errors, and asynchronous HTTP 202 handle preservation.
apps/sim/tools/snowflake/sql.test.ts Exercises SQL generation, validation, escaping, binding limits, and operation-specific statements.
apps/sim/tools/registry.ts Registers the new Snowflake tools for workflow execution.

Sequence Diagram

sequenceDiagram
  participant Workflow
  participant Tool as Snowflake Tool
  participant Transport as Shared Tool Transport
  participant API as Snowflake SQL API
  Workflow->>Tool: Execute configured operation
  Tool->>Transport: Build bounded authenticated request
  Transport->>API: Submit SQL API request
  alt Completed response
    API-->>Transport: 2xx result
    Transport->>Tool: Transform response
    Tool-->>Workflow: Result partition and DML statistics
  else Asynchronous response
    API-->>Transport: HTTP 202 with statement handle
    Transport->>Tool: Transform pending response
    Tool-->>Workflow: RUNNING and statement handle
    Workflow->>Tool: Get Statement
    Tool->>API: Poll handle and partition
    API-->>Workflow: Completed result
  end
Loading

Reviews (9): Last reviewed commit: "fix(snowflake): emit task history time b..." | Re-trigger Greptile

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot]
greptile-apps Bot reviewed Aug 8, 2026
View reviewed changes
Comment thread apps/sim/tools/snowflake/utils.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot]
greptile-apps Bot reviewed Aug 8, 2026
View reviewed changes
Comment thread apps/sim/tools/snowflake/utils.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 910c5b5. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@vercel
vercel Bot deployed to Preview August 8, 2026 05:31
View deployment
@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the feat/snowflake-integration branch from 910c5b5 to d2092b2 Compare August 8, 2026 05:38
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit d2092b2. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@vercel
vercel Bot deployed to Preview August 8, 2026 05:43
View deployment
waleedlatif1
waleedlatif1 reviewed Aug 8, 2026
View reviewed changes
Comment thread apps/sim/tools/index.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the feat/snowflake-integration branch from d2092b2 to d2e1968 Compare August 8, 2026 06:47
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot]
greptile-apps Bot reviewed Aug 8, 2026
View reviewed changes
Comment thread apps/sim/tools/snowflake/utils.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot deployed to Preview August 8, 2026 06:53
View deployment
cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit d2e1968. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Files Created / Modified: (All Standard)

apps/
├── docs/
│ ├── components/
│ │ └── ui/
│ │ └── icon-mapping.ts
│ └── content/
│ └── docs/
│ └── en/
│ └── integrations/
│ ├── meta.json
│ └── snowflake.mdx
│
└── sim/
├── blocks/
│ ├── blocks/
│ │ └── snowflake.ts
│ └── registry-maps.ts
│
├── lib/
│ └── integrations/
│ ├── icon-mapping.ts
│ └── integrations.json
│
└── tools/
├── generated/
│ ├── tool-ids.ts
│ ├── tool-metadata.ts
│ └── tool-outputs.ts
│
├── snowflake/
│ ├── index.ts
│ ├── types.ts
│ ├── utils.ts
│ ├── utils.test.ts
│ ├── sql.ts
│ ├── sql.test.ts
│ ├── execute_sql.ts
│ ├── get_statement.ts
│ ├── cancel_statement.ts
│ ├── insert_rows.ts
│ ├── update_rows.ts
│ ├── upsert_rows.ts
│ ├── delete_rows.ts
│ ├── load_data.ts
│ ├── list_warehouses.ts
│ ├── get_warehouse.ts
│ ├── resume_warehouse.ts
│ ├── suspend_warehouse.ts
│ ├── list_tasks.ts
│ ├── get_task.ts
│ ├── run_task.ts
│ ├── list_task_runs.ts
│ ├── get_task_run.ts
│ ├── cancel_task_run.ts
│ ├── get_task_run_output.ts
│ ├── introspect_schema.ts
│ └── call_procedure.ts
│
└── registry.ts

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Bill Leoutsakos Bill Leoutsakos
feat(integrations): add Snowflake PAT integration
2deca33
@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the feat/snowflake-integration branch from d2e1968 to 2deca33 Compare August 8, 2026 08:14
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit b6405b0. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Bill Leoutsakos Bill Leoutsakos
refactor(integrations): simplify Snowflake safeguards
624b737
@vercel
vercel Bot deployed to Preview August 8, 2026 17:04
View deployment
Bill Leoutsakos Bill Leoutsakos
refactor(snowflake): isolate statement capabilities
97d237c
@vercel
vercel Bot deployed to Preview August 8, 2026 20:41
View deployment
Bill Leoutsakos Bill Leoutsakos
fix(snowflake): localize required user agent
1603852
@vercel
vercel Bot temporarily deployed to Preview August 8, 2026 20:45 Inactive
Bill Leoutsakos Bill Leoutsakos
chore(snowflake): limit changes to integration scope
22fde60
@BillLeoutsakosvl346

BillLeoutsakosvl346 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Snowflake integration diff against staging

38 files changed: 29 created, 9 modified, 0 deleted

[A] Created
[M] Modified

apps/
├── docs/
│ ├── components/ui/
│ │ └── icon-mapping.ts [M]
│ └── content/docs/en/integrations/
│ ├── meta.json [M]
│ └── snowflake.mdx [A]
└── sim/
├── blocks/
│ ├── blocks/
│ │ └── snowflake.ts [A]
│ └── registry-maps.ts [M]
├── lib/integrations/
│ ├── icon-mapping.ts [M]
│ └── integrations.json [M]
└── tools/
├── generated/
│ ├── tool-ids.ts [M]
│ ├── tool-metadata.ts [M]
│ └── tool-outputs.ts [M]
├── snowflake/
│ ├── call_procedure.ts [A]
│ ├── cancel_statement.ts [A]
│ ├── cancel_task_run.ts [A]
│ ├── delete_rows.ts [A]
│ ├── execute_sql.ts [A]
│ ├── get_statement.ts [A]
│ ├── get_task.ts [A]
│ ├── get_task_run.ts [A]
│ ├── get_task_run_output.ts [A]
│ ├── get_warehouse.ts [A]
│ ├── index.ts [A]
│ ├── insert_rows.ts [A]
│ ├── introspect_schema.ts [A]
│ ├── list_task_runs.ts [A]
│ ├── list_tasks.ts [A]
│ ├── list_warehouses.ts [A]
│ ├── load_data.ts [A]
│ ├── resume_warehouse.ts [A]
│ ├── run_task.ts [A]
│ ├── sql.test.ts [A]
│ ├── sql.ts [A]
│ ├── suspend_warehouse.ts [A]
│ ├── types.ts [A]
│ ├── update_rows.ts [A]
│ ├── upsert_rows.ts [A]
│ ├── utils.test.ts [A]
│ └── utils.ts [A]
└── registry.ts [M]

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot deployed to Preview August 8, 2026 21:02
View deployment
waleedlatif1 added 2 commits August 8, 2026 16:05
@waleedlatif1
fix(snowflake): correct SQL generation, transport, and param conventions
c4d3718
Address defects found by validation against the Snowflake SQL API v2 and
SQL reference docs.

SQL generation:
- lift PARSE_JSON out of the VALUES clause into a projecting SELECT; the
  previous form is rejected for any object or array value
- escape backslashes as well as quotes in string literals, closing a COPY
  option injection through the user-or-llm stagePath and pattern fields
- reject "--" in stage paths, which commented out every following clause
- emit COPY INTO clauses in the documented positional order
- exclude only view types in introspect_schema so temporary, external, and
  event tables are visible
- use plain equality in MERGE and reject null or duplicate match keys
- bound rows and bound-value bytes for every statement, measured in UTF-8
- reject qualified task names, which TASK_HISTORY silently ignores
- replace a raw NUL byte in the source with its escape sequence

Transport:
- read DML stats from the documented top-level ResultSet property
- drop Link-header and 391908 paging, which belong to the retired API, and
  report partition completeness as unknown rather than falsely complete
- require a 2xx status before trusting a success SQLSTATE
- cap response bodies and fail closed on invalid session context names

Conventions:
- inline shared params into each tool instead of cross-file spreads, which
  also lets the docs generator emit host and apiKey
- use the official Snowflake brand mark on a white tile
@waleedlatif1
Merge remote-tracking branch 'origin/staging' into feat/snowflake-int…
04e9a87
…egration

# Conflicts:
#	apps/sim/tools/generated/tool-ids.ts
#	apps/sim/tools/generated/tool-metadata.ts
#	apps/sim/tools/generated/tool-outputs.ts
@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 04e9a87. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@vercel
vercel Bot deployed to Preview August 8, 2026 23:12
View deployment
@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@cursor review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes
Comment thread apps/sim/blocks/blocks/snowflake.ts Resolved

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1
fix(snowflake): emit task history time bounds as literals
36b3aa3
TASK_HISTORY only accepts bind variables for RESULT_LIMIT and TASK_NAME per
BCR-1410, and that change explicitly excludes a bind passed through another
function first. A bind in SCHEDULED_TIME_RANGE_START/END is therefore dropped
without an error, so the requested window became a no-op and the function fell
back to returning the most recent runs. Emit validated literals instead, which
also restores Snowflake's seven-day range error.

Also reject a fractional skip-file percentage at the block boundary rather than
in the builder, and correct the cancel description: a cancelled child marks the
task graph run failed, so downstream tasks are skipped rather than continuing.
@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1

waleedlatif1 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

@cursor review

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Aug 8, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 36b3aa3. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@waleedlatif1
waleedlatif1 merged commit 3096de8 into staging Aug 8, 2026
29 of 30 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@waleedlatif1
waleedlatif1 deleted the feat/snowflake-integration branch August 8, 2026 23:27
@waleedlatif1 waleedlatif1 mentioned this pull request Aug 8, 2026
v0.7.64: snowflake, dynatrace, mintlify, files and security hardening, perf improvements #6460
Merged
@vercel
vercel Bot deployed to Preview August 8, 2026 23:28
View deployment
This was referenced Aug 8, 2026
chore(snowflake): drop the local write caps in favor of the shared limit #6461
Merged
feat(snowflake): credential-based auth, object pickers, and 9 new operations #6474
Merged

This branch was successfully deployed

1 active deployment
Preview — 36b3aa39 Deployed Aug 8, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@waleedlatif1 waleedlatif1 waleedlatif1 left review comments
@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cursor cursor[bot] cursor[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@BillLeoutsakosvl346 @waleedlatif1
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.