Skip to content

docs(#3271): update boost README with project description, plugin structure, and prerequisites#3274

Open
fullsend-ai-coder[bot] wants to merge 2 commits into
mainfrom
agent/3271-update-boost-readme
Open

docs(#3271): update boost README with project description, plugin structure, and prerequisites#3274
fullsend-ai-coder[bot] wants to merge 2 commits into
mainfrom
agent/3271-update-boost-readme

Conversation

@fullsend-ai-coder
Copy link
Copy Markdown

Replace the scaffolded placeholder README with meaningful content:

  • Expanded project description explaining boost as a rewrite of augment
    (AI assistant plugin for Backstage)
  • Planned plugin structure table listing boost, boost-backend, and
    boost-common packages
  • Prerequisites section specifying Node 22+ and yarn

Closes #3271

Post-script verification

  • Branch is not main/master (agent/3271-update-boost-readme)
  • Secret scan passed (gitleaks — 44b418c47c8b8b90afdfd86d8d8fe433b985fc35..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

…ucture, and prerequisites

Replace the scaffolded placeholder README with meaningful content:
- Expanded project description explaining boost as a rewrite of augment
  (AI assistant plugin for Backstage)
- Planned plugin structure table listing boost, boost-backend, and
  boost-common packages
- Prerequisites section specifying Node 22+ and yarn

Closes #3271

Signed-off-by: fullsend-code <fullsend-code@users.noreply.github.com>
@fullsend-ai-coder fullsend-ai-coder Bot requested review from a team, durandom and gabemontero as code owners June 2, 2026 10:29
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 2, 2026

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@fullsend-ai-review
Copy link
Copy Markdown

fullsend-ai-review Bot commented Jun 2, 2026

Review

Findings

Medium

Low

  • [naming-convention] workspaces/boost/README.md — The sibling Augment workspace README hyperlinks each plugin name to its local directory path (e.g., [@red-hat-developer-hub/backstage-plugin-augment](./plugins/augment/)). The Boost PR lists plugin names as plain text. This is understandable since the plugin directories don't exist yet, but the convention gap should be addressed when the plugins are created.

  • [pattern-inconsistency] workspaces/boost/README.md — The PR adds a preamble sentence ("The following plugins are planned:") before the plugin table. No other workspace README uses a preamble — the Augment README goes directly from the ## Plugins heading to the table. This may be intentional to signal planned (vs. existing) plugins.

Previous run

Review

Findings

Medium

  • [naming-convention] workspaces/boost/README.md:5 — The section heading ## Planned Plugin Structure deviates from the established convention. The sibling workspace augment (which boost is explicitly a rewrite of) and other workspaces use ## Plugins as the heading. Using a non-standard heading breaks consistency across workspace READMEs.
    Remediation: Rename the heading to ## Plugins. If the plugins are not yet published, a note under the heading (e.g., The following plugins are planned:) can convey the same intent without changing the heading convention.

  • [pattern-inconsistency] workspaces/boost/README.md:7 — The plugin table uses three columns (Plugin, Package, Description) with short names and unlinked plain-text package names. The established pattern in augment/README.md uses a two-column table (Plugin, Description) where column 1 contains the full scoped package name rendered as a Markdown link to the plugin directory (e.g., [@red-hat-developer-hub/backstage-plugin-augment](./plugins/augment/)).
    Remediation: Adopt the two-column format from augment: column 1 should contain the linked package name, column 2 the description.

Low

  • [code-organization] workspaces/boost/README.md:14 — A standalone ## Prerequisites section listing Node.js and Yarn requirements is added, but no other workspace README in the repository includes such a section. Repository-wide prerequisites are typically documented at the root level.
    Remediation: Consider removing this section to stay consistent with other workspace READMEs, or defer to the repository root README for shared prerequisites.

  • [intent-alignment] workspaces/boost/README.md:17 — The prerequisites section states "Node.js 22 or later" but the workspace's package.json engines field specifies "22 || 24", which is more restrictive (it does not allow Node 23 or 25+). The README phrasing could mislead contributors into thinking any version >= 22 is supported.
    Remediation: Update to match the engines field, e.g., "Node.js 22 or 24 (see engines in package.json)".

@fullsend-ai-review fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Jun 2, 2026
@durandom
Copy link
Copy Markdown
Member

durandom commented Jun 2, 2026

/fs-fix

- Rename heading from "Planned Plugin Structure" to "Plugins" to match
  convention used by augment and other workspace READMEs
- Switch plugin table to two-column format (Plugin, Description) matching
  the established augment pattern
- Add "planned" note to convey pre-release status without a non-standard heading
- Remove Prerequisites section for consistency with other workspace READMEs

Addresses review feedback on #3274

Signed-off-by: fullsend-fix <fullsend-code@users.noreply.github.com>
@fullsend-ai-coder
Copy link
Copy Markdown
Author

🔧 Fix agent — iteration 1 (human-triggered)

Addressed all 4 review findings: renamed section heading to match convention, adopted two-column plugin table format from augment, and removed Prerequisites section (which also resolved the Node.js version mismatch).

Fixed (4):

  1. naming-convention: heading deviates from established convention (workspaces/boost/README.md): Renamed heading from 'Planned Plugin Structure' to 'Plugins' to match the convention used by augment and other workspace READMEs. Added a 'The following plugins are planned:' note to convey pre-release status.
  2. pattern-inconsistency: plugin table format differs from augment (workspaces/boost/README.md): Switched from three-column table (Plugin, Package, Description) to two-column table (Plugin, Description) matching the augment README pattern. Plugin names use the full scoped package name. Links omitted since plugin directories do not exist yet.
  3. code-organization: standalone Prerequisites section not used in other workspaces (workspaces/boost/README.md): Removed the Prerequisites section entirely to stay consistent with other workspace READMEs. Repository-wide prerequisites are documented at the root level.
  4. intent-alignment: Node.js version in README doesn't match engines field (workspaces/boost/README.md): Resolved by removing the Prerequisites section (finding 3). The inaccurate version statement no longer exists.

Tests: passed

Updated by fullsend fix agent

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

@fullsend-ai-review fullsend-ai-review Bot added requires-manual-review Review requires human judgment and removed requires-manual-review Review requires human judgment labels Jun 2, 2026
Copy link
Copy Markdown
Contributor

@gabemontero gabemontero left a comment

Choose a reason for hiding this comment

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

just some FYI's @durandom

This workspace contains the Boost plugin family for Red Hat Developer Hub.

Boost is a rewrite of the [Augment](../augment/) plugin.
Boost is a rewrite of the [Augment](../augment/) plugin — an AI assistant plugin for Backstage that provides RAG-powered document search, multi-agent orchestration, tool calling via MCP servers, and safety guardrails, all running against open-source models on your own infrastructure.
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.

this is fine for now but fyi @durandom I plan on updating this when I submit the proposal / spec / background doc

| --------------------------------------------------------- | ------------------------------------------------ |
| @red-hat-developer-hub/backstage-plugin-boost | Frontend UI — chat interface and agent status |
| @red-hat-developer-hub/backstage-plugin-boost-backend | Backend service — API routes, orchestration logic |
| @red-hat-developer-hub/backstage-plugin-boost-common | Shared types, constants, and utilities |
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.

similarly we'll be expanding this beyond what augment is currently providing @durandom ... I'll deal with that as well as part of the proposal / spec /etc/ PR

@gabemontero
Copy link
Copy Markdown
Contributor

any idea @durandom if fullsend's agents can address this from the CI:

Run yarn prettier:check
Checking formatting...
[warn] README.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.
Error: Process completed with exit code 1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-manual-review Review requires human judgment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update boost README with plugin roadmap and development setup

2 participants