Skip to content

[codex] add cost-aware skill dogfood report#46

Merged
hudsonaikins merged 1 commit into
mainfrom
dogfood/profitctl-cost-aware-skill
May 29, 2026
Merged

[codex] add cost-aware skill dogfood report#46
hudsonaikins merged 1 commit into
mainfrom
dogfood/profitctl-cost-aware-skill

Conversation

@hudsonaikins
Copy link
Copy Markdown
Contributor

Summary

  • Add a dogfood report for the ProfitCtl cost-aware Codex skill.
  • Capture repo-grounded scenario evidence for Workers vs Vercel, AgentOS Cloud Run vs Workers, and Exa Research Pro cost risk.
  • Link the report from the docs index.

Validation

  • go test ./...
  • go run scripts/judge_cost_standards.go
  • python3 /Users/hudson/.codex/skills/.system/skill-creator/scripts/quick_validate.py /Users/hudson/.codex/skills/profitctl-cost-aware
  • python3 /Users/hudson/.codex/skills/.system/skill-creator/scripts/quick_validate.py /Users/hudson/Documents/GitHub/IntelIP/ProfitCtl/skills/profitctl-cost-aware

Notes

This is documentation-only. It keeps template assumptions low/medium confidence and records that scenarios are not invoice-grade until calibrated with telemetry or invoices.

@hudsonaikins hudsonaikins marked this pull request as ready for review May 29, 2026 23:29
@hudsonaikins hudsonaikins merged commit cd2e095 into main May 29, 2026
3 checks passed
@hudsonaikins hudsonaikins deleted the dogfood/profitctl-cost-aware-skill branch May 29, 2026 23:29
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

Here are some automated review suggestions for this pull request.

Reviewed commit: dec3533fe0

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +11 to +15
- `web-app/package.json`: Next.js 16, OpenNext Cloudflare, Clerk, Stripe, Postgres client, PostHog.
- `web-app/wrangler.jsonc`: Cloudflare Worker deployment with OpenNext assets, Images binding, self-service binding, and 3000 ms CPU limit.
- `Condere/deployment/cloudbuild.yaml`: AgentOS production runtime deploys to Cloud Run Gen2, 1 vCPU, 2 GiB, concurrency 10, min instances 0, max instances 3, private IAM.
- `Condere/condere_src_os/core/cloud_run_production_plan.py`: repo cost expectations say Cloud Run idle scales near zero, steady single instance is `$57.02/month`, peak three-instance cap is `$171.07/month`, before request, egress, build, and free-tier offsets.
- `Condere/condere_src_os/infra/observability/run_cost_estimation.py`: Exa default estimates are `$0.007` quick search, `$0.022` search with summary/deep/answer, `$0.002` URL content, `$0.10` research task, `$0.25` research pro task.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Correct the non-repo evidence paths

When readers follow this new Product Docs report, the core “Repo Context Inspected” evidence is not reproducible in this repository: I checked rg --files for these exact paths and none of web-app/package.json, web-app/wrangler.jsonc, Condere/deployment/cloudbuild.yaml, cloud_run_production_plan.py, or run_cost_estimation.py exist under /workspace/ProfitCtl. Because the report’s cost recommendations are explicitly grounded in those files, this makes the documented Workers/Cloud Run/Exa conclusions look repo-backed when the cited inputs are absent; please either include the correct in-repo evidence paths or label this as an external-repo dogfood run with links/commit identifiers.

Useful? React with 👍 / 👎.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant