Skip to content

fix: polish prompt wizard styling and model catalog onboarding#95

Merged
Mr-Dark-debug merged 2 commits intomainfrom
mr-dark-debug/fix-sharp-edges-and-dropdown-styling
Oct 9, 2025
Merged

fix: polish prompt wizard styling and model catalog onboarding#95
Mr-Dark-debug merged 2 commits intomainfrom
mr-dark-debug/fix-sharp-edges-and-dropdown-styling

Conversation

@Mr-Dark-debug
Copy link
Copy Markdown
Collaborator

Summary

  • deepen the neo-brutalist field system with curved gradients and refreshed shadows, and apply the new treatment to the prompt upload wizard inputs/buttons
  • surface provider and family guidance within the admin model manager and add an overview call-to-action for building the AI model catalog
  • tighten the custom select arrow positioning to match the updated rounded inputs

Testing

  • npm run lint

https://chatgpt.com/codex/tasks/task_e_68e78cc8914c832dbeeff16c12c4a90f

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 9, 2025

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

Project Deployment Preview Comments Updated (UTC)
syntax-blogs Error Error Oct 9, 2025 11:08am

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

models:prompt_models (is_primary, ai_models (id, display_name, category, version)),
tags:prompt_tags_junction (prompt_tags (id, name, category)),
comment_count:prompt_comments(count)
`
type PromptQueryRow = Database['public']['Tables']['prompts']['Row'] & {
user: {
id: string
display_name: string | null
avatar_url: string | null
} | null
assets: Database['public']['Tables']['prompt_assets']['Row'][] | null
models: Array<{
is_primary: boolean
ai_models: Database['public']['Tables']['ai_models']['Row'] | null
}> | null
tags: Array<{
prompt_tags: Database['public']['Tables']['prompt_tags']['Row'] | null
}> | null
comment_count: Array<{ count: number | null }> | null
}
const formatPromptSummary = (row: PromptQueryRow): PromptSummary => {
const models = (row.models ?? [])
.map((item) => item.ai_models)
.filter((model): model is Database['public']['Tables']['ai_models']['Row'] => Boolean(model))
.map((model) => ({
id: model.id,
display_name: model.display_name,
category: model.category,
version: model.version ?? null,

P1 Badge Populate provider metadata when constructing prompt summaries

The commit updates PromptSummary.models to include category_id, provider, and family, but selectPromptFields and the mapping in formatPromptSummary still fetch only id, display_name, category, and version. As written, every prompt summary returned from getPrompts will have those new fields undefined, so any UI that expects the provider/family guidance introduced in this change cannot render it. Running a strict type-check will also fail because the objects produced here are missing the newly required properties. The query and mapper should select and pass through the additional columns.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 9, 2025

Deployment failed with the following error:

Resource is limited - try again in 3 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/prashant-project?upgradeToPro=build-rate-limit

@Mr-Dark-debug Mr-Dark-debug merged commit 628dfc1 into main Oct 9, 2025
1 check failed
@Mr-Dark-debug Mr-Dark-debug deleted the mr-dark-debug/fix-sharp-edges-and-dropdown-styling branch October 9, 2025 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant