Skip to content

docs: add /tinybird skill for pipe and datasource deploys - #4572

Open
emlimlf wants to merge 1 commit into
mainfrom
docs/tinybird-skill
Open

docs: add /tinybird skill for pipe and datasource deploys#4572
emlimlf wants to merge 1 commit into
mainfrom
docs/tinybird-skill

Conversation

@emlimlf

@emlimlf emlimlf commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a /tinybird skill documenting the edit-and-deploy cycle for Tinybird pipes/datasources (venv setup, workspace selection between lfx_insights_stg/lfx_insights, format, push, PR, prod rollout gated on explicit confirmation)
  • Registers the skill in .claude/rules/skill-guidance.md with trigger phrases

Context

This was originally opened as linuxfoundation/insights#2087, but the Tinybird resources it documents (.pipe/.datasource, format.sh, the README) all live in this repo under services/libs/tinybird/, not in insights. Moving the skill here removes the need for it to locate a sibling checkout or guard against editing a submodule — it now just operates directly on this repo's own Tinybird resources and PR conventions.

Test plan

  • Confirm the skill loads in a fresh Claude Code session in this repo
  • Confirm it prompts for confirmation before any tb push to the lfx_insights (prod) workspace

Note: no JIRA (CM) key included in this title yet — pr-title-jira-key-lint will fail until one is added.

Signed-off-by: Efren Lim <elim@linuxfoundation.org>
Copilot AI balanced review requested due to automatic review settings September 8, 2026 00:16
@cursor

cursor Bot commented Sep 8, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only changes to Claude skill guidance; no application or Tinybird runtime code is modified.

Overview
Introduces a /tinybird Claude Code skill for editing and deploying Tinybird pipes and datasources under services/libs/tinybird/, and wires it into skill routing so assistants suggest it for tb push/tb pull, pipe/datasource changes, and staging vs prod workspace work.

The new skill spells out an end-to-end flow (venv, auth handoff, workspace selection for lfx_insights_stg / lfx_insights, tb pull, format.sh, staging push, PR conventions, post-merge prod push) with guardrails: no prod tb push without explicit confirmation, no datasource deletes or tb push --populate, and safe handling of .tinyb credentials via jq only. It explicitly defers backfills, Docker testing, and architecture deep-dives to the existing Tinybird README.

Reviewed by Cursor Bugbot for commit 10063a7. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a /tinybird skill for safely editing and deploying Tinybird resources.

Changes:

  • Documents setup, staging, PR, and production workflows.
  • Registers Tinybird-related trigger phrases.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
.claude/skills/tinybird/SKILL.md Defines the deployment workflow and safeguards.
.claude/rules/skill-guidance.md Registers /tinybird routing triggers.
Suppressed comments (1)

.claude/skills/tinybird/SKILL.md:126

  • The production path omits the explicit current-workspace verification required by lines 70–71 and demonstrated for staging. Add the check here so the workspace shown during confirmation is provably lfx_insights before the production push.
tb workspace use lfx_insights

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +34 to +36
cd services/libs/tinybird
source .venv/bin/activate
tb --version # expect 5.x — ignore the "upgrade to 6.x" nag, that's a different product line
Comment on lines +65 to +67
tb workspace ls
tb workspace use lfx_insights_stg # staging/dev
tb workspace use lfx_insights # production
`lfx_insights_stg`:

```bash
tb pull --force --match <resource_name>
Comment on lines +112 to +113
in parens at the end, `--signoff -S`, and the PR title must carry the CM key (CI-enforced by
`pr-title-jira-key-lint.yml`). Note: this repo's Tinybird CI (`tinybird-ci.yml`) only checks
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.

3 participants