FF: restructure and edit#3312
Conversation
🛡️ Jit Security Scan Results
✅ No security findings were detected in this PR
Security scan by Jit
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b5c5666. Configure here.
| --- | ||
|
|
||
| Redis Feature Form supports multiple providers, secrets provider management, and workspaces. | ||
|
|
There was a problem hiding this comment.
Emptied page leaves stale title and broken link
Medium Severity
All body content was moved out of streaming.md to register-providers.md and manage-workspace.md, but the frontmatter still says title: Providers and workspaces and description: Build stream-backed features with Kafka, streaming transformations, and Redis serving.. This renders as an empty page with a misleading title in site navigation. Additionally, overview.md links to this file as "Connect providers" — that link now lands on a blank page. The frontmatter needs updating (or the file needs a redirect/removal).
Reviewed by Cursor Bugbot for commit b5c5666. Configure here.
| description: | ||
| linkTitle: | ||
| weight: 1 | ||
| --- |
There was a problem hiding this comment.
Page with content has empty frontmatter fields
Medium Severity
manage-workspace.md contains real body content (workspace management commands moved from streaming.md) but its title, description, and linkTitle frontmatter fields are all blank. Unlike the other new placeholder files that are intentionally empty stubs, this page has substantive content. The missing frontmatter means it will render with no page title in navigation and no heading, and register-providers.md already links to it.
Reviewed by Cursor Bugbot for commit b5c5666. Configure here.
|
|
||
| Before you register providers, make sure you have: | ||
|
|
||
| - A workspace. See [Manage workspaces](./manage-workspace.md) for the workspace lifecycle commands. |
There was a problem hiding this comment.
Raw markdown links instead of Hugo relref shortcodes
Medium Severity
The two cross-page links in register-providers.md use raw Markdown relative links (./manage-workspace.md, ./define-and-deploy-features.md) instead of Hugo relref shortcodes. Every other cross-page link in this project uses {{< relref "..." >}}, and there is no custom render-link.html hook in the layouts to rewrite .md paths to proper URLs. These links will be broken in the rendered site because Hugo will pass the .md href through as-is, which won't match the generated page URLs.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit b5c5666. Configure here.
dwdougherty
left a comment
dwdougherty
left a comment
There was a problem hiding this comment.
Apart from the issues already identified by Bugbot, LGTM.
2 participants
Note
Low Risk
Low-risk documentation-only restructure, but several new pages contain empty front matter (e.g., missing
title/linkTitle) which could affect site navigation/build output.Overview
Moves the provider/secret-provider/workspace CLI documentation out of
streaming.mdinto new dedicated pages: a fullregister-providers.mdguide and a smallermanage-workspace.mdreference.Adds several new FeatureForm docs stubs (
concepts.md,configure-auth.md,define-and-deploy-features.md,query-data.md,reference.md,serve-features.md,update-features.md) that currently only include front matter placeholders, and leavesstreaming.mdeffectively empty aside from its header.Reviewed by Cursor Bugbot for commit b5c5666. Bugbot is set up for automated code reviews on this repo. Configure here.