Skip to content

Remove "hosting" wording from recipe copy - #14

Open
batuhan wants to merge 1 commit into
masterfrom
no-hosting-copy
Open

batuhan wants to merge 1 commit into
masterfrom
no-hosting-copy

Conversation

@batuhan

@batuhan batuhan commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Spacefast should not be described with the word "hosting".

Changed:

  • recipes/status: "API / hosting product" is now "API or developer platform"; "hosting company" is now "platform company" (prompt and meta).
  • recipes/wp-react meta: "static hosting" is now "a static site".

Kept:

  • recipes/party and recipes/hackathon: "who's hosting" / "is hosting" mean hosting an event.
  • recipes/report: fictional survey data about hosting popularity, narrow third-party content.

bun test scripts/catalog.test.mjs passes.

View in Indent View in Slack
Tag @indent to continue the conversation here.


Get evidence
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.

Summary by CodeRabbit

  • Documentation
    • Updated the status page example to describe an API or developer platform.
    • Clarified that the SPA fallback supports deep links on a static site. These wording updates align the examples and descriptions with the terminology used in the recipes. They do not change the recipes’ behavior.

@indent

indent Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Issues

No issues found.

CI Checks

All CI checks passed on 3f72bcd.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No issues found in this copy-only change.
Full results

Indent Review Agent

  • Summary: No issues found in this copy-only change.
  • Last ran on commit: 3f72bcd2
  • Latest result
    {
      "summary": "No issues found in this copy-only change.",
      "findings": []
    }

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The status recipe text now describes an API or developer platform. The WP React recipe describes its SPA fallback as supporting deep links on a static site.

Changes: Status recipe

Layer / File(s) Summary
Status recipe wording
recipes/status/prompt.md, recipes/status/meta.json
The prompt and based_on description refer to an API or developer platform.

WP React recipe

Layer / File(s) Summary
SPA fallback wording
recipes/wp-react/meta.json
The sections description changes “static hosting” to “a static site.”

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Possibly related PRs

  • spacefast/examples#12: This PR removes recipes/wp-react/meta.json, which this change edits. The changes may conflict depending on merge order.
Merge Risk: Minimal · up to 3f72b

This PR only rewords recipe descriptions to avoid calling Spacefast "hosting". It changes no behavior, so there is no meaningful merge risk.

Architecture risk: Low · up to 3f72b

The change affects 1 system.

Changed systems: recipes

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — recipes (service) was modified; 3 changed files map to changed impact.

Before / after behavior

  • observed — Modified behavior in recipes/status/meta.json: The based_on description replaces “hosting company” with “platform company.”
  • observed — Modified behavior in recipes/status/prompt.md: The prompt changes the target from an “API / hosting product” status page to an “API or developer platform” status page.
  • observed — Modified behavior in recipes/wp-react/meta.json: The SPA fallback description changes “static hosting” to “a static site.”
Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "Remove "hosting" wording from recipe copy" matches the main change. The PR replaces "hosting" wording in the recipes/status prompt and metadata and in the recipes/wp-react metadata. The tit…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "Remove "hosting" wording from recipe copy" matches the main change. The PR replaces "hosting" wording in the recipes/status prompt and metadata and in the recipes/wp-react metadata. The title names the change and its scope. The PR keeps "hosting" in recipes/party, recipes/hackathon, and recipes/report on purpose, but the title does not claim otherwise, so it is an acceptable summary.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 974ff08f-f4e7-4b63-9f53-7cb5e3d5af0f
Commits

Reviewing files that changed from the base of the PR and between 0117b0a and 3f72bcd.

Files selected for processing: 3

  • recipes/status/meta.json
  • recipes/status/prompt.md
  • recipes/wp-react/meta.json

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

This branch has not been deployed

No deployments
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