Skip to content

improvement(planetscale): add placeholders to every input and fill missing docs intros - #8447

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/planetscale-block-polish
Sep 30, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
fix/planetscale-block-polish

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • PlanetScale block: every text input and selector now has a placeholder (service token ID and token had none, nor did most advanced fields); formatted fields show their shape (timestamps, ranges) and optional fields name their default
  • Added the MANUAL-CONTENT intro section to the PlanetScale, Bitbucket, Modal, and Otter docs pages, the only generated integration pages missing one
  • add-integration and validate-integration skills now require a placeholder on every text-entry and selector subBlock, and a docs intro section (with a template)

Type of Change

  • Improvement

Testing

  • bun run scripts/generate-docs.ts: no generated changes, and all four intros survive regeneration; docs:check passes
  • bun run lint, block registry check, bun run check:audits, docs-manifest:check

Checklist

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

🤖 Generated with Claude Code

@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Building Building Preview Sep 30, 2026 3:16am UTC

Request Review

@waleedlatif1
waleedlatif1 merged commit 460c01f into staging Sep 30, 2026
22 of 23 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/planetscale-block-polish branch September 30, 2026 03:17
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Low risk] Documentation and UI placeholder text updates.

The PR appears safe to merge, with two non-blocking copy corrections worth making.

Findings

  1. P2 Major version hint omits exceptions ▶
  2. P2 Modal token requirement overstated ▶

Summary

The PR adds PlanetScale input hints, hand-written introductions for four integration pages, and placeholder and intro requirements to the integration-authoring skills.

  • The new hints and introductions largely match the existing integration contracts.
  • Two pieces of copy need qualification: PlanetScale's major-version default and Modal's token requirement.

Reviews (1) · Last reviewed commit: "improvement(planetscale): add placeholde..."

id: 'majorVersion',
title: 'Major Version',
type: 'short-input',
placeholder: 'Defaults to the parent branch version',

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.

P2 Major version hint omits exceptions The placeholder says the version defaults to the parent branch version. For a branch restored from a backup, PlanetScale ignores this setting; when there is no parent branch, it can instead use the database default branch's version. Someone configuring a restore could rely on a version choice that will not take effect.

Knowledge Base Used: Integrations, connectors, and tools

- **Generate completions**: Get chat completions from a model you serve on a Modal Endpoint
- **List models**: See which models a token can reach

To connect, create a proxy auth token in your Modal workspace settings and enter its token ID and secret in the block.

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.

P2 Modal token requirement overstated This instruction tells readers to create and enter a proxy token to connect, but calling an unauthenticated Web Function requires neither token field. Readers who only want to call a public function are directed to set up credentials they do not need. Distinguish that case from endpoint operations that require a token.

Knowledge Base Used: Integrations, connectors, and tools

This branch was successfully deployed

1 active deployment
Preview — 601eff22 Deployed Sep 30, 2026 by vercel[bot]
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