Skip to content

feat: add Deploy with Tilde example state - #17

Merged
danielblignaut merged 6 commits into
mainfrom
codex/reconcile-example-workspace-state
Aug 7, 2026
Merged

danielblignaut merged 6 commits into
mainfrom
codex/reconcile-example-workspace-state

Conversation

@danielblignaut

@danielblignaut danielblignaut commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • rename each template state file to tilde.state.yaml
  • add variable-driven ChatKit endpoint configuration
  • use provider-managed GitHub App provisioning and required integration setup
  • embed the public Deploy with Tilde button in every example README

Validation

  • code-review bot: pnpm lint
  • code-review bot: pnpm typecheck
  • code-review bot: production pnpm build with placeholder required environment values
  • Sentry remediation bot: pnpm lint
  • Sentry remediation bot: pnpm typecheck
  • Sentry remediation bot: pnpm test
  • Sentry remediation bot: production pnpm build with placeholder required environment values

Dependency

Merge after the API deploy route and /deploy-button.svg are available in production.

@danielblignaut danielblignaut changed the title Add Deploy to Tilde example state feat: add Deploy with Tilde example state Aug 6, 2026
@danielblignaut
danielblignaut marked this pull request as ready for review August 6, 2026 15:08
@greptile-apps

greptile-apps Bot commented Aug 6, 2026

Copy link
Copy Markdown

Greptile Summary

The PR renames both portable state files to tilde.state.yaml, parameterizes their ChatKit endpoints, and adds Deploy with Tilde links. It also expands the imported provider, credential, reverse-proxy, and MCP resources.

  • Adds endpoint variables and updates README import instructions.
  • Adds GitHub channel and reverse-proxy resources to the code-review example.
  • Adds reverse proxies and human-approval tooling to the Sentry remediation example.
  • The Sentry one-click deployment still requires an edit that the button flow cannot perform.

Confidence Score: 4/5

The Sentry deploy-button path should be fixed before merging because it imports an unusable webhook signing secret.

The direct deployment imports the committed placeholder as the Sentry signing key, preventing webhook verification and therefore preventing remediation sessions from starting.

Files Needing Attention: sentry-remediation-bot/README.md and sentry-remediation-bot/tilde.state.yaml

Important Files Changed

Filename Overview
code-review-bot/README.md Updates state-file references and documents the endpoint variable and deploy-button workflow.
code-review-bot/tilde.state.yaml Parameterizes the agent endpoint and adds internally consistent GitHub channel and reverse-proxy resources.
sentry-remediation-bot/README.md Adds a direct deploy button, but the linked state still requires manually replacing a webhook secret before import.
sentry-remediation-bot/tilde.state.yaml Parameterizes the endpoint and expands resources, while retaining a non-variable Sentry webhook-secret placeholder that breaks direct deployment.

Reviews (1): Last reviewed commit: "docs: embed Deploy with Tilde buttons" | Re-trigger Greptile


## Setup

[![Deploy with Tilde](https://api.trytilde.ai/deploy-button.svg)](https://api.trytilde.ai/deploy?repository-url=https%3A%2F%2Fgithub.com%2Ftrytilde%2Fexamples&state-path=sentry-remediation-bot%2Ftilde.state.yaml)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Deploy imports placeholder signing secret

When a user follows this deploy button, Tilde imports the committed state file directly with REPLACE_WITH_SENTRY_APP_CLIENT_SECRET as its webhook signing key, causing Sentry webhook verification to fail and preventing remediation sessions from starting.

@danielblignaut
danielblignaut merged commit 8f3876a into main Aug 7, 2026
3 checks passed
@danielblignaut
danielblignaut deleted the codex/reconcile-example-workspace-state branch August 7, 2026 08:03
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