feat(gcp-to-aws): AI clarify detect-confirm cards + AI-only mini assumption sheet - #174
Merged
icarthick merged 5 commits intoJul 22, 2026
Merged
Conversation
… question, AI-only mini sheet, Q27 separation
Clean merge to current main (through awslabs#173; clarify-ai.md co-touched, no overlap). dprint clean, full build green.
icarthick
added a commit
to herosjourney/startups
that referenced
this pull request
Jul 22, 2026
Clean merge to current main (through awslabs#174; estimate-infra.md + generate-artifacts-report.md co-touched by awslabs#173, no overlap). dprint clean, full build green.
added 2 commits
July 22, 2026 16:19
…ify-ai.md presentation rule (resolves seam with awslabs#170)
Contributor
Author
|
Pushed 731cdba (after merging current main, which now includes #170): the batch-composition line #170 added to clarify.md still listed Q27 inside the AI/agentic essentials batch, contradicting this PR's Q27 separate-follow-up rule in clarify-ai.md. The batch list now excludes Q27 and points at the presentation rule, so an agent reading both files gets one consistent instruction. Lint green (855 files). |
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The problem, in plain English: The plugin's code scanner usually already knows which AI framework and models a startup uses — but the moment detection is anything less than fully certain, the questions fall back to cold walls of options: a 7-option framework taxonomy and a 19-option model list (Gemini 3.5 Flash Thinking vs Gemini 3.1 Flash-Lite vs GPT-5.4 Nano…). A founder who just watched the plugin correctly find
langchainin their requirements.txt is then asked to pick their framework from scratch, as if the scan never happened. And in the AI-only flow there's no assumption sheet at all — the full Clarify wizard shows users what was detected before asking anything, but AI-only users go straight into question batches.Changes:
clarify-ai.md): when detection produced signals but not full resolution, present "I detected LangChain (requirements.txt) — that's right / edit / just LangChain" instead of the full taxonomy. The full option list only appears cold when detection found nothing.clarify-ai.md): detected-but-uncertain models are offered as a short pick-list of the actual candidates. When nothing was detected, ask provider family first (Gemini / GPT / o-series / other), then show only the matching subset — never the 19-option list in one message. The full list remains the interpretation catalog.clarify-ai-only.md, new Step 1.75): detected values (framework, model, capabilities) and to-be-assumed defaults presented as a compact confirm-or-edit sheet before Batch 1, mirroring the full wizard's sheet-before-questions contract. Sheet-resolved questions are skipped in the batches.clarify-ai.md): Activate-credits question asked as its own short follow-up after the technical essentials ("Last one, and it's about money rather than tech"), never mixed into the technical batch. Existing anti-inference rules (no eligibility from spend, no silent default) untouched.Multi-workload confirmation already had accept/edit/drop actions — unchanged.
Why
Usability review: detect-confirm is the natural completion of the extraction machinery this skill already has (
chosen_by: "extracted", confidence scores, auto-skip rules). The only missing piece was what to do at partial confidence, which is the common case.Test plan
mise run lint:md— 0 errorsmodels[0].confidence = 0.6produces a two-candidate confirm card, not the full catalogOut of scope