Skip to content

Conversation

@Dustinturner44
Copy link

Summary

Brief description of what this PR does and why.

Fixes #(issue)

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

How has this been tested? What should reviewers focus on?

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)

Screenshots/Videos

waleedlatif1 and others added 16 commits October 11, 2025 22:23
…oai#1608)

* improvement(performance): remove unused source/target indices, add index on snapshot id (simstudioai#1603)

* fix(blog): rename building to blogs with redirect (simstudioai#1604)

* improvement(privacy-policy): updated privacy policy for google (simstudioai#1602)

* updated privacy policy for google

* update terms, privacy, and emails to incl address and update verbiage

* feat(guardrails): added guardrails block/tools and docs (simstudioai#1605)

* Adding guardrails block

* ack PR comments

* cleanup checkbox in dark mode

* cleanup

* fix supabase tools

* fix(inference-billing): fix inference billing when stream is true via API, add drag-and-drop functionality to deployed chat (simstudioai#1606)

* fix(inference): fix inference billing when stream is true via API

* add drag-and-drop to deployed chat

* feat(mistal): added mistral as a provider, updated model prices (simstudioai#1607)

* feat(mistal): added mistral as a provider, updated model prices

* remove the ability for a block to reference its own outluts

* fixed order of responses for guardrails block

* feat(versions): added the ability to rename deployment versions (simstudioai#1610)

* fix(vulns): fix various vulnerabilities and enhanced code security (simstudioai#1611)

* fix(vulns): fix SSRF vulnerabilities

* cleanup

* cleanup

* regen docs

* remove unused deps

* fix failing tests

* cleanup

* update deps

* regen bun lock
)

* fix(debug-mode): remove duplicate debug mode flag (simstudioai#1714)

* feat(i18n): update translations (simstudioai#1709)

* improvement(condition): added variable and envvar highlighting for condition input (simstudioai#1718)

* fix(dashboard): add additional context for paginated logs in dashboard, add empty state when selected cell has no data (simstudioai#1719)

* fix(dashboard): add additional context for paginated logs in dashboard, add empty state when selected cell has no data

* apps/sim

* renaming

* remove relative import

* feat(tools): added webflow OAuth + tools (simstudioai#1720)

* feat(tools): added webflow OAuth + tools

* remove itemId from delete item

* remove siteId

* added webhook triggers + oauth scopes + site/collection selector

* update sample payload for webflow triggers

* cleanup

* fix discord color

* feat(i18n): update translations (simstudioai#1721)

* improvement(schedule): fix UI bug with schedule modal (simstudioai#1722)
* fix(already-cancelled-sub): UI should allow restoring subscription

* restore functionality fixed

* fix
@vercel
Copy link

vercel bot commented Nov 10, 2025

@Dustinturner44 is attempting to deploy a commit to the Sim Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Overview

Greptile Summary

This PR renames devcontainer.json to devcontainer.json. by adding a trailing period to the filename.

  • Critical Issue: The trailing period breaks the devcontainer specification. VS Code and other devcontainer-compatible tools require the file to be named exactly devcontainer.json without any trailing characters.
  • Impact: This change will prevent the devcontainer from being recognized, breaking the development environment setup for all developers using devcontainers.
  • Recommendation: This PR should not be merged. If the intent was to disable the devcontainer, consider moving it to a backup location or documenting the reason for disabling it.

Confidence Score: 0/5

  • This PR will break devcontainer functionality and should not be merged
  • Score of 0 reflects a critical breaking change: renaming devcontainer.json with a trailing period violates the devcontainer specification and will prevent VS Code and other tools from recognizing the devcontainer configuration
  • .devcontainer/devcontainer.json. must be renamed back to devcontainer.json (without trailing period)

Important Files Changed

File Analysis

Filename Score Overview
.devcontainer/devcontainer.json. 0/5 File renamed with trailing period, breaking devcontainer specification compliance

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant VSC as VS Code
    participant DC as Devcontainer Engine
    participant File as .devcontainer/devcontainer.json.
    
    Dev->>VSC: Open repository
    VSC->>DC: Request devcontainer config
    DC->>File: Look for devcontainer.json
    File--xDC: File not found (wrong name)
    DC--xVSC: No valid devcontainer config
    VSC--xDev: Devcontainer not available
Loading

Additional Comments (1)

  1. .devcontainer/devcontainer.json., line 1-37 (link)

    logic: This file rename breaks devcontainer functionality. VS Code and other devcontainer tools require the file to be named exactly devcontainer.json, not devcontainer.json. (with trailing period). This will prevent the devcontainer from being recognized and loaded.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

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.

4 participants