Skip to content

Add AI project price evaluation#4

Merged
TUPM96 merged 3 commits into
mergeos-bounties:masterfrom
hummusonrails:bounty/ai-price-evaluation
May 26, 2026
Merged

Add AI project price evaluation#4
TUPM96 merged 3 commits into
mergeos-bounties:masterfrom
hummusonrails:bounty/ai-price-evaluation

Conversation

@hummusonrails
Copy link
Copy Markdown
Contributor

@hummusonrails hummusonrails commented May 26, 2026

Related to #3

Summary

  • Add a structured project price evaluation service and authenticated API endpoint at POST /api/projects/evaluate-price
  • Return suggested price, editable range, confidence, task/category breakdown, assumptions, and risks
  • Connect the project setup wizard's AI budget estimator to the API so users can apply the suggested budget before publishing

Validation

  • go test ./...
  • npm test from frontend/

The estimator is deterministic and keeps the pricing logic outside the UI layer so it can be tested and later swapped to an external AI provider without exposing secrets client-side.

@TUPM96
Copy link
Copy Markdown
Contributor

TUPM96 commented May 26, 2026

Thanks for the update. Since this PR changes the project setup wizard / AI budget estimator UI, could you add visual evidence for reviewers, such as screenshots or a short video/GIF showing the estimator flow and the suggested budget being applied? That will make it easier to verify the UI behavior alongside the listed tests.

@TUPM96 TUPM96 added evidence: missing PR needs screenshot, GIF, video, or other visual evidence. star: missing PR author must star this repository before bounty review. bounty: feature Feature or enhancement bounty work. labels May 26, 2026
@hummusonrails
Copy link
Copy Markdown
Contributor Author

Thanks for the review @TUPM96 — adding visual evidence below.

AI Budget Estimator flow

The GIF walks through Step 3 (Budget & timeline) of the project setup wizard and shows:

  1. Initial state — Budget input at the user's manually-entered $10,000 (Fixed price), with the static "AI Budget Suggestion" rail showing the legacy $8,000 - $12,000 heuristic.
  2. Estimate with AI clicked — The new POST /api/projects/evaluate-price endpoint returns a structured suggestion. The wizard renders:
    • Suggested price: $7,000 with editable range $5,950 - $8,400
    • Confidence: medium · editable before publishing
    • Breakdown: Base scope $2,200, Deliverables $1,800, Technical surface $1,000, Complexity $700
    • Risk note: "Major scope changes after publishing can move the price range."
  3. Use estimate clicked — Budget input flips to Range mode with $7,000 applied; the right-hand Project summary rail updates to Budget: $7,000 (Range).

The estimator is deterministic (same input → same output) so the numbers above are reproducible from the scope/deliverables/tech-stack values entered in steps 1–2. Tests covering the breakdown math are in backend/internal/core/price_evaluation_test.go.

Happy to add a second pass capturing the wizard from Step 1 if that would help.

@hummusonrails hummusonrails marked this pull request as ready for review May 26, 2026 07:29
@github-actions github-actions Bot added evidence: provided PR includes acceptable screenshot, GIF, video, or other visual evidence. and removed evidence: missing PR needs screenshot, GIF, video, or other visual evidence. labels May 26, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 26, 2026

PR readiness check.

  • Evidence: provided.
  • Repository star: verified for @hummusonrails.
  • Bounty type: feature bounty.

All bounty readiness checks currently pass.

@github-actions
Copy link
Copy Markdown

PR readiness check.

  • Evidence: provided.
  • Repository star: missing for @hummusonrails. Please star this repository before claiming or continuing bounty work.
  • Bounty type: feature bounty.

After updating the PR, rerun this check or add a PR comment so it can recheck.

…valuation

# Conflicts:
#	frontend/src/App.vue
hummusonrails added a commit to hummusonrails/mergeos that referenced this pull request May 26, 2026
Mark the AI project evaluation submission as open (not draft), with
evidence provided. Repository star is still pending.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@hummusonrails
Copy link
Copy Markdown
Contributor Author

Re-triggering the readiness check — repo is now starred, evidence GIF posted above.

@hummusonrails
Copy link
Copy Markdown
Contributor Author

Quick note on the star: missing label — the repo has been starred. Confirmable via the public stargazers endpoint:

$ gh api repos/mergeos-bounties/mergeos/stargazers --jq '.[].login'
TUPM96
hummusonrails

The readiness workflow calls GET /users/{username}/starred/{owner}/{repo}, which per the GitHub REST docs is the authenticated-user endpoint — it returns 404 for any user other than the one whose token is running the workflow. The same 404 is reproducible unauthenticated:

$ curl -s -o /dev/null -w "%{http_code}\n" https://api.github.com/users/hummusonrails/starred/mergeos-bounties/mergeos
404

That's why both this PR and PR #5 are flagged star: missing despite the stargazers list containing both authors. A drop-in fix is to paginate GET /repos/{owner}/{repo}/stargazers and check membership — happy to send a small follow-up PR against the workflow if useful.

Mark the AI project evaluation submission as open (not draft), with
evidence provided. Repository star is still pending.
@hummusonrails hummusonrails force-pushed the bounty/ai-price-evaluation branch from 5751425 to c80b7eb Compare May 26, 2026 07:40
hummusonrails added a commit to hummusonrails/mergeos that referenced this pull request May 26, 2026
@TUPM96
Copy link
Copy Markdown
Contributor

TUPM96 commented May 26, 2026

LGTM.

@github-actions github-actions Bot added star: verified PR author has starred this repository. and removed star: missing PR author must star this repository before bounty review. labels May 26, 2026
@TUPM96 TUPM96 merged commit 4406a84 into mergeos-bounties:master May 26, 2026
1 check failed
@TUPM96
Copy link
Copy Markdown
Contributor

TUPM96 commented May 26, 2026

MergeOS approved and merged this PR.

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

Labels

bounty: feature Feature or enhancement bounty work. evidence: provided PR includes acceptable screenshot, GIF, video, or other visual evidence. star: verified PR author has starred this repository.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants