feat: add prompt deployment view to settings panel with updated styling by impoiler · Pull Request #2352 · maximhq/bifrost

impoiler · 2026-03-28T06:57:05Z

Summary

Integrates the prompt deployments enterprise feature into the UI by adding the PromptDeploymentView component to the settings panel and updating its visual styling.

Changes

Added PromptDeploymentView component to the prompts settings panel to display enterprise prompt deployment information
Updated PromptDeploymentView styling to use a more compact card-like design with top alignment, smaller icon, and border styling
Added enterprise UI source path to CSS configuration to include enterprise component styles

Type of change

Affected areas

How to test

Verify the prompt deployments enterprise feature appears in the settings panel with proper styling.

# UI
cd ui
pnpm i || npm i
pnpm test || npm test
pnpm build || npm run build

Navigate to the prompts page and open the settings panel to verify the PromptDeploymentView component renders correctly with the updated card-style layout.

Screenshots/Recordings

The PromptDeploymentView now appears as a compact card in the settings panel instead of a full-height centered view.

Breaking changes

Yes
No

Related issues

Related to enterprise prompt deployments feature integration.

Security considerations

No security implications - this is a UI enhancement for displaying enterprise feature information.

Checklist

I read docs/contributing/README.md and followed the guidelines
I added/updated tests where appropriate
I updated documentation where needed
I verified builds succeed (Go and UI)
I verified the CI pipeline passes locally if applicable

coderabbitai · 2026-03-28T06:57:11Z

📝 Walkthrough

Summary by CodeRabbit

New Features
- Added a prompt deployment view inside Settings for direct access.
Style
- Updated the deployment view to be full-width and top-aligned with refined spacing, rounded corners, border and padding for clearer presentation.
- Adjusted icon sizing and stroke for improved visual balance.
Chores
- Expanded style scanning to include additional UI sources so utilities are generated.

Walkthrough

Inserted PromptDeploymentView into SettingsPanel; modified outer wrapper, ContactUsView alignment/styling, and Router icon sizing in the enterprise fallback prompt view; and added a Tailwind @source entry to include external enterprise TSX files.

Changes

Cohort / File(s)	Summary
Layout & Styling Updates `ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx`	Removed `h-full` from root wrapper (`h-full w-full` → `w-full`); replaced `ContactUsView`'s `className="mx-auto min-h-[80vh]"` with `align="top"` and `className="justify-start gap-3 rounded-md border p-4"`; updated `Router` icon sizing from `className="h-[5.5rem] w-[5.5rem]"` (strokeWidth=1) to `className="h-8 w-8"` (strokeWidth=1.5).
Tailwind Content Scanning `ui/app/globals.css`	Added `@source` directive for `../../../bifrost-enterprise/ui/*/.tsx` so Tailwind includes those TSX files in content scanning/class extraction.
Settings Panel Integration `ui/components/prompts/fragments/settingsPanel.tsx`	Imported `PromptDeploymentView` and appended a standalone `<PromptDeploymentView />` at the end of the settings content within the `ScrollArea`; no other logic or props changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 I nudged a view into the fold,
Trimmed a height and gave it bold,
Icons neat and borders new,
Tailwind spies the enterprise view,
Hops of code, a tiny gold.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: adding the prompt deployment view component to the settings panel with updated styling.
Description check	✅ Passed	The description covers all major required sections including summary, changes, type of change, affected areas, testing instructions, and breaking changes confirmation.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch 03-28-feat_prompt-deployment-strategies-fallback-screens

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

impoiler · 2026-03-28T06:57:20Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

coderabbitai

♻️ Duplicate comments (1)

ui/app/globals.css (1)
6-6: ⚠️ Potential issue | 🟠 Major

Stylelint still needs Tailwind v4 at-rule allowlisting for Line 6.

This looks like the same unresolved blocker previously flagged: @source is valid for Tailwind v4, but scss/at-rule-no-unknown will fail unless source is whitelisted in stylelint config.
#!/bin/bash
# Verify stylelint rule configuration for Tailwind v4 at-rules (read-only).
# Expected: scss/at-rule-no-unknown includes "source" in ignoreAtRules.

fd -HI 'stylelint.config.*|.stylelintrc.*' | while read -r f; do
  echo "---- $f ----"
  rg -n -C3 'scss/at-rule-no-unknown|ignoreAtRules|source|tailwind|apply|theme|utility|variant|custom-variant|plugin|reference' "$f"
done
As per coding guidelines, "always check the stack if there is one for the current PR. do not give localized reviews for the PR, always see all changes in the light of the whole stack of PRs (if there is a stack, if there is no stack you can continue to make localized suggestions/reviews)".
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@ui/app/globals.css` at line 6, The `@source` at-rule on line referencing
"@source" in ui/app/globals.css is valid in Tailwind v4 but will be flagged by
the scss/at-rule-no-unknown rule; update your Stylelint config
(stylelint.config.* or .stylelintrc.*) to add "source" to the
scss/at-rule-no-unknown.ignoreAtRules array (or equivalent ignoreAtRules
setting) so the `@source` at-rule is allowlisted, and re-run the verification
script provided to confirm "source" appears in ignoreAtRules.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@ui/app/globals.css`:
- Line 6: The `@source` at-rule on line referencing "@source" in
ui/app/globals.css is valid in Tailwind v4 but will be flagged by the
scss/at-rule-no-unknown rule; update your Stylelint config (stylelint.config.*
or .stylelintrc.*) to add "source" to the scss/at-rule-no-unknown.ignoreAtRules
array (or equivalent ignoreAtRules setting) so the `@source` at-rule is
allowlisted, and re-run the verification script provided to confirm "source"
appears in ignoreAtRules.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3365a1f1-d4ce-45e4-af31-364e9352e98c

📥 Commits

Reviewing files that changed from the base of the PR and between 2b88043 and 8ab486e.

📒 Files selected for processing (3)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx
ui/app/globals.css
ui/components/prompts/fragments/settingsPanel.tsx

✅ Files skipped from review due to trivial changes (2)

ui/components/prompts/fragments/settingsPanel.tsx
ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ui/app/globals.css`:
- Line 6: Update the `@source` import in ui/app/globals.css to point at the
enterprise component folder; replace the incorrect
"../../../bifrost-enterprise/ui/**/*.tsx" value used in the `@source` declaration
with the correct relative path "../app/_fallbacks/enterprise/**/*.tsx" so
Tailwind will scan the enterprise component .tsx files.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9bc44ff3-4d45-45d2-9e4c-e6e4246256d4

📥 Commits

Reviewing files that changed from the base of the PR and between 8ab486e and 1a73c1b.

📒 Files selected for processing (3)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx
ui/app/globals.css
ui/components/prompts/fragments/settingsPanel.tsx

✅ Files skipped from review due to trivial changes (1)

ui/components/prompts/fragments/settingsPanel.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx

coderabbitai

🧹 Nitpick comments (1)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx (1)

8-9: Drop redundant justify-start from caller className.

align="top" already maps to justify-start inside ContactUsView, so keeping both duplicates layout responsibility across components.

Suggested cleanup

 			<ContactUsView
 				align="top"
-				className="justify-start gap-3 rounded-md border p-4"
+				className="gap-3 rounded-md border p-4"
 				icon={<Router className="h-8 w-8" strokeWidth={1.5} />}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx`
around lines 8 - 9, Remove the redundant "justify-start" utility from the
className on the caller component in promptDeploymentView.tsx (the element with
className="justify-start gap-3 rounded-md border p-4") because align="top"
already causes ContactUsView to apply justify-start; keep the other utilities
(gap-3, rounded-md, border, p-4) intact so only "justify-start" is deleted to
avoid duplicate layout responsibility between the caller and ContactUsView.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx`:
- Around line 8-9: Remove the redundant "justify-start" utility from the
className on the caller component in promptDeploymentView.tsx (the element with
className="justify-start gap-3 rounded-md border p-4") because align="top"
already causes ContactUsView to apply justify-start; keep the other utilities
(gap-3, rounded-md, border, p-4) intact so only "justify-start" is deleted to
avoid duplicate layout responsibility between the caller and ContactUsView.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 54b510b0-2609-422d-818e-b244529716e9

📥 Commits

Reviewing files that changed from the base of the PR and between 684cf74 and 5465c4e.

📒 Files selected for processing (3)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx
ui/app/globals.css
ui/components/prompts/fragments/settingsPanel.tsx

✅ Files skipped from review due to trivial changes (1)

ui/components/prompts/fragments/settingsPanel.tsx

coderabbitai

♻️ Duplicate comments (1)

ui/app/globals.css (1)
6-6: ⚠️ Potential issue | 🟡 Minor

This issue was already flagged in a previous review.

The path ../../../bifrost-enterprise/ui/**/*.tsx was verified to not exist (see past review verification script results).

Additionally, this line appears redundant: line 4 already scans ../app/**/*.tsx, which includes both app/enterprise (symlinked enterprise source per vite.config.mts) and app/_fallbacks/enterprise (fallback implementations). The component mentioned in the PR summary (promptDeploymentView.tsx) is located in the fallbacks directory and is already covered by line 4's glob pattern.

Recommendation: Remove this line entirely, as the existing @source directive at line 4 already covers enterprise component files.
🗑️ Proposed fix: remove the redundant line
 `@source` "../app/**/*.tsx";
 `@source` "../node_modules/streamdown/dist/*.js";
-@source "../../../bifrost-enterprise/ui/**/*.tsx";
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@ui/app/globals.css` at line 6, Remove the redundant `@source` directive that
references "../../../bifrost-enterprise/ui/**/*.tsx" from ui/app/globals.css;
the existing `@source` "../app/**/*.tsx" already covers enterprise and fallback
components (including promptDeploymentView.tsx), so delete the line containing
the "../../../bifrost-enterprise/ui/**/*.tsx" `@source` directive to avoid
referencing a non-existent path.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@ui/app/globals.css`:
- Line 6: Remove the redundant `@source` directive that references
"../../../bifrost-enterprise/ui/**/*.tsx" from ui/app/globals.css; the existing
`@source` "../app/**/*.tsx" already covers enterprise and fallback components
(including promptDeploymentView.tsx), so delete the line containing the
"../../../bifrost-enterprise/ui/**/*.tsx" `@source` directive to avoid referencing
a non-existent path.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3fc4321-0778-4ca6-a18d-4ef25ee559ad

📥 Commits

Reviewing files that changed from the base of the PR and between 5465c4e and 3df2fd7.

📒 Files selected for processing (3)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx
ui/app/globals.css
ui/components/prompts/fragments/settingsPanel.tsx

✅ Files skipped from review due to trivial changes (1)

ui/components/prompts/fragments/settingsPanel.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

ui/app/_fallbacks/enterprise/components/prompt-deployments/promptDeploymentView.tsx

akshaydeo · 2026-04-16T16:34:23Z

Merge activity

Apr 16, 4:34 PM UTC: A user started a stack merge that includes this pull request via Graphite.
Apr 16, 4:38 PM UTC: @akshaydeo merged this pull request with Graphite.

The base branch was changed.

impoiler mentioned this pull request Mar 28, 2026

refactor: deleted subpages from the prompt repo #2327

Merged

18 tasks

impoiler changed the title ~~feat/prompt-deployment-strategies-fallback-screens~~ feat: add prompt deployment view to settings panel with updated styling Mar 28, 2026

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from f1f9a6a to e604a81 Compare March 30, 2026 04:57

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from ad44de0 to d1ea419 Compare March 30, 2026 05:58

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch 2 times, most recently from 2452907 to ec4a3c5 Compare March 30, 2026 08:10

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from d1ea419 to 3627ca6 Compare March 30, 2026 08:10

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from ec4a3c5 to d139c6c Compare March 30, 2026 10:34

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 3627ca6 to 07c974f Compare March 30, 2026 10:34

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from d139c6c to a9b323a Compare March 30, 2026 11:13

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from 8bdc181 to a512710 Compare March 30, 2026 16:46

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from 00cb164 to e634f42 Compare March 31, 2026 04:58

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from 3932074 to d97fd81 Compare April 1, 2026 07:34

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch 2 times, most recently from 36a9229 to 2e358b7 Compare April 1, 2026 08:43

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from d97fd81 to 96c07d5 Compare April 1, 2026 08:43

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from 2e358b7 to ea4750f Compare April 1, 2026 14:46

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from 805694b to 576eb0a Compare April 2, 2026 02:31

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch 2 times, most recently from 28f364c to 671d7af Compare April 3, 2026 07:05

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from a6207ca to 66f0442 Compare April 3, 2026 08:22

impoiler force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from 671d7af to b1d2ef0 Compare April 3, 2026 08:22

impoiler force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 66f0442 to c69e6bd Compare April 6, 2026 05:35

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from a7aa89a to a4751ee Compare April 13, 2026 18:17

coderabbitai Bot reviewed Apr 13, 2026

View reviewed changes

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from a4751ee to 10865dd Compare April 13, 2026 19:15

roroghost17 force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 8ab486e to 1a73c1b Compare April 13, 2026 19:15

coderabbitai Bot requested changes Apr 13, 2026

View reviewed changes

Comment thread ui/app/globals.css

coderabbitai Bot previously approved these changes Apr 13, 2026

View reviewed changes

roroghost17 force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 1a73c1b to 9f01c3f Compare April 14, 2026 10:17

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch 2 times, most recently from 0a82df2 to aac5b15 Compare April 15, 2026 20:19

roroghost17 force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch 2 times, most recently from 2c77290 to 684cf74 Compare April 16, 2026 08:05

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from aac5b15 to 38a5bde Compare April 16, 2026 08:05

roroghost17 mentioned this pull request Apr 16, 2026

docs: updates docs for prompts plugin #2763

Merged

18 tasks

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from 38a5bde to f139ac8 Compare April 16, 2026 14:50

roroghost17 force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 684cf74 to 5465c4e Compare April 16, 2026 14:50

coderabbitai Bot reviewed Apr 16, 2026

View reviewed changes

impoiler added 5 commits April 16, 2026 21:20

Add support for variables in prompt messages

d3fbfdb

Add support for variables in the prompt messages backend

a1f8922

refactor moved CELRuleBuilder to lib for reusability

f538e6e

refactor/remove prompt repo subpages

ee19296

feat/prompt-deployment-strategies-fallback-screens

3df2fd7

roroghost17 force-pushed the 03-27-refactor_remove_prompt_repo_subpages branch from f139ac8 to ee19296 Compare April 16, 2026 15:52

roroghost17 force-pushed the 03-28-feat_prompt-deployment-strategies-fallback-screens branch from 5465c4e to 3df2fd7 Compare April 16, 2026 15:52

coderabbitai Bot reviewed Apr 16, 2026

View reviewed changes

akshaydeo changed the base branch from 03-27-refactor_remove_prompt_repo_subpages to graphite-base/2352 April 16, 2026 16:37

akshaydeo changed the base branch from graphite-base/2352 to v1.5.0 April 16, 2026 16:38

akshaydeo merged commit 27019f1 into v1.5.0 Apr 16, 2026
4 of 10 checks passed

akshaydeo deleted the 03-28-feat_prompt-deployment-strategies-fallback-screens branch April 16, 2026 16:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add prompt deployment view to settings panel with updated styling#2352

feat: add prompt deployment view to settings panel with updated styling#2352
akshaydeo merged 5 commits intov1.5.0from
03-28-feat_prompt-deployment-strategies-fallback-screens

impoiler commented Mar 28, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Mar 28, 2026 •

edited

Loading

Summary by CodeRabbit

Walkthrough

Changes

Estimated code review effort

Poem

❌ Failed checks (1 warning)

Uh oh!

impoiler commented Mar 28, 2026 •

edited by roroghost17

Loading

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot left a comment

Uh oh!

akshaydeo commented Apr 16, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

impoiler commented Mar 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Type of change

Affected areas

How to test

Screenshots/Recordings

Breaking changes

Related issues

Security considerations

Checklist

Uh oh!

coderabbitai Bot commented Mar 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Walkthrough

Changes

Estimated code review effort

Poem

❌ Failed checks (1 warning)

Uh oh!

impoiler commented Mar 28, 2026 • edited by roroghost17 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

akshaydeo commented Apr 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

impoiler commented Mar 28, 2026 •

edited

Loading

coderabbitai Bot commented Mar 28, 2026 •

edited

Loading

impoiler commented Mar 28, 2026 •

edited by roroghost17

Loading

akshaydeo commented Apr 16, 2026 •

edited

Loading