v0.3.7: ms teams webhook, docker fixes, condition block dropdown, fixed routing with workflow block #743

waleedlatif1 · 2025-07-22T00:16:47Z

fix(docker): fixed docker container healthchecks, added instructions to README for pgvector (fix(docker): fixed docker container healthchecks, added instructions to README for pgvector #735)
fix(config): add t3 to list of transpiled packages to ensure that envvars are picked up correctly (fix(config): add t3 to list of transpiled packages to ensure that envvars are picked up correctly #737)
fix(condition): fixed condition block dropdown (fix(condition): fixed condition block dropdown #738)
working impl fo ms teams outgoing webhook (feat(webhooks): ms teams webhook #740)
fix(router): fixed routing issue with workflow block, added tests (fix(router): fixed routing issue with workflow block, added tests #739)
fix(teams-webhook): response json needs type (fix(teams-webhook): response json needs type #741)

…to README for pgvector (#735) * fixed docker container healthchecks * add additional instructions for pgvector extension to README

…vars are picked up correctly (#737)

* fixed routing issue with workflow block, added tests * add new dmeo * change console to logger * new demo * changed all console to logger

vercel · 2025-07-22T00:16:52Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sim (staging)	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 22, 2025 0:16am

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Skipped (Inspect)			Jul 22, 2025 0:16am

greptile-apps

Greptile Summary

This PR (v0.3.7) introduces several significant improvements to the codebase:

Docker Container Healthchecks: Healthcheck endpoints have been updated to use dedicated /health endpoints instead of root paths, making container health monitoring more reliable and explicit.
Environment Variable Handling: Added t3-oss packages to transpilePackages in Next.js config to ensure proper handling of environment variables (particularly NEXT_PUBLIC_SOCKET_URL) in production builds.
Microsoft Teams Integration: Implemented comprehensive Microsoft Teams webhook support with proper HMAC signature verification and response typing, following the same patterns as existing webhook providers.
Workflow Routing Architecture: Significantly improved routing logic by properly categorizing workflow blocks as flow control blocks, ensuring correct execution path selection and preventing unintended block executions.
Standardized Logging: Replaced console.log statements throughout the codebase with structured logging using createLogger, improving debugging capabilities and production observability.

Confidence score: 4/5

The changes appear stable and well-tested, but need careful monitoring during initial deployment due to workflow routing changes.
Score of 4 given due to comprehensive test coverage and clear architectural improvements, but the broad scope of changes (especially routing logic) warrants careful deployment monitoring.
Key files needing extra attention:
- apps/sim/executor/routing/routing.ts
- apps/sim/blocks/blocks/webhook.ts
- docker-compose.prod.yml

_{34 files reviewed, 6 comments}

_{Edit Code Review Bot Settings | Greptile}

greptile-apps · 2025-07-22T00:17:20Z

apps/sim/lib/utils.test.ts

-})
+vi.mock('@/lib/env', () => ({
+  env: {
+    ENCRYPTION_KEY: '0123456789abcdef0123456789abcdef0123456789abcdef0123456789abcdef',


style: consider defining the encryption key in a test constants file for reuse across test files

greptile-apps · 2025-07-22T00:17:22Z

apps/sim/executor/tests/multi-input-routing.test.ts

+              systemPrompt: 'return the following in urdu roman english',
+              userPrompt: '<function1.result>\n<function2.result>',
+              model: 'gpt-4o',


logic: Agent's userPrompt references undefined variables - should be '<function-1.result>' and '<function-2.result>' to match block IDs

Suggested change

systemPrompt: 'return the following in urdu roman english',

userPrompt: '<function1.result>\n<function2.result>',

model: 'gpt-4o',

systemPrompt: 'return the following in urdu roman english',

userPrompt: '<function-1.result>\n<function-2.result>',

model: 'gpt-4o',

greptile-apps · 2025-07-22T00:17:23Z

apps/sim/app/api/webhooks/trigger/[path]/route.ts

+      // Get the raw body for HMAC verification
+      const rawBody = await request.text()


logic: Request body is consumed twice - once here and previously in the initial parsing. This could cause issues.

Suggested change

// Get the raw body for HMAC verification

const rawBody = await request.text()

// Use the already parsed rawBody for HMAC verification

// const rawBody = await request.text() // This would fail since body already consumed

greptile-apps · 2025-07-22T00:17:23Z

apps/sim/executor/tests/multi-input-routing.test.ts

+
+    // CRITICAL ASSERTIONS:
+    expect(function1DepMet).toBe(true) // Executed and active
+    expect(function2DepMet).toBe(true) // Not in active path, so considered met (line 1151)


syntax: Comment references non-existent line 1151. Update comment to reference correct line number or remove line reference

greptile-apps · 2025-07-22T00:17:23Z

...rkflow-block/components/sub-block/components/webhook/components/providers/microsoftteams.tsx

+  testResult: {
+    success: boolean
+    message?: string
+    test?: any


style: Avoid using 'any' type for test property. Consider defining a specific type for test results.

greptile-apps · 2025-07-22T00:17:24Z

...rkflow-block/components/sub-block/components/webhook/components/providers/microsoftteams.tsx

+const teamsWebhookExample = JSON.stringify(
+  {
+    type: 'message',
+    id: '1234567890',
+    timestamp: '2023-01-01T00:00:00.000Z',
+    localTimestamp: '2023-01-01T00:00:00.000Z',
+    serviceUrl: 'https://smba.trafficmanager.net/amer/',
+    channelId: 'msteams',
+    from: {
+      id: '29:1234567890abcdef',
+      name: 'John Doe',
+    },
+    conversation: {
+      id: '19:meeting_abcdef@thread.v2',
+    },
+    text: 'Hello Sim Studio Bot!',
+  },
+  null,
+  2
+)


style: Consider moving webhook example payload to a separate config file to keep component focused on UI logic

delve-auditor · 2025-07-22T00:20:58Z

✅ No security or compliance issues detected. Reviewed everything up to e4fbb67.

Security Overview

🔎 Scanned files: 35 changed file(s)

Detected Code Changes

Change Type	Relevant files
Bug Fix	► route.ts Add typed response for Teams webhook ► routing.ts Update router decision handling ► executor/index.ts Modify dependency checking for routed paths
Enhancement	► microsoftteams.tsx Add Microsoft Teams webhook provider ► webhook.ts Implement Teams webhook support ► utils.ts Add Teams webhook validation and processing
Refactor	► console-logger.ts Replace console.log with structured logging

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

waleedlatif1 and others added 6 commits July 20, 2025 21:27

fix(docker): fixed docker container healthchecks, added instructions …

8007601

…to README for pgvector (#735) * fixed docker container healthchecks * add additional instructions for pgvector extension to README

fix(config): add t3 to list of transpiled packages to ensure that env…

d919073

…vars are picked up correctly (#737)

fix(condition): fixed condition block dropdown (#738)

d94bfd9

working impl fo ms teams outgoing webhook (#740)

7739917

fix(router): fixed routing issue with workflow block, added tests (#739)

7f0f902

* fixed routing issue with workflow block, added tests * add new dmeo * change console to logger * new demo * changed all console to logger

fix(teams-webhook): response json needs type (#741)

e4fbb67

greptile-apps bot reviewed Jul 22, 2025

View reviewed changes

waleedlatif1 merged commit 560d184 into main Jul 22, 2025
8 of 9 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

v0.3.7: ms teams webhook, docker fixes, condition block dropdown, fixed routing with workflow block #743

v0.3.7: ms teams webhook, docker fixes, condition block dropdown, fixed routing with workflow block #743

Uh oh!

waleedlatif1 commented Jul 22, 2025

Uh oh!

vercel bot commented Jul 22, 2025

Uh oh!

greptile-apps bot left a comment

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

greptile-apps bot Jul 22, 2025

Uh oh!

Uh oh!

delve-auditor bot commented Jul 22, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		// Get the raw body for HMAC verification
		const rawBody = await request.text()

v0.3.7: ms teams webhook, docker fixes, condition block dropdown, fixed routing with workflow block #743

v0.3.7: ms teams webhook, docker fixes, condition block dropdown, fixed routing with workflow block #743

Uh oh!

Conversation

waleedlatif1 commented Jul 22, 2025

Uh oh!

vercel bot commented Jul 22, 2025

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Greptile Summary

Confidence score: 4/5

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Jul 22, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

delve-auditor bot commented Jul 22, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants