feat: recommend lightweight Comet workflow paths - #248
Open
googs1025 wants to merge 3 commits into
Open
Conversation
Reviewer's GuideAdds a configurable Classic workflow intensity and typed lightweight route recommendations to the Comet intent runtime, wires that configuration into the Classic CLI and generated runtimes, and updates the /comet-open skill contracts and tests so small, low‑risk requests can choose tweak/hotfix before proceeding with full OpenSpec workflows. Sequence diagram for lightweight workflow recommendation and confirmationsequenceDiagram
actor User
participant CometClassicCLI as comet-classic
participant ProjectConfig as readClassicWorkflowIntensity
participant IntentRuntime as resolveCometIntentRoute
participant CometOpenSkill as /comet-open
User->>CometClassicCLI: classic-intent route (JSON input)
CometClassicCLI->>CometClassicCLI: withConfiguredWorkflowIntensity(input)
CometClassicCLI->>ProjectConfig: readClassicWorkflowIntensity()
ProjectConfig-->>CometClassicCLI: ClassicWorkflowIntensityValue
CometClassicCLI->>IntentRuntime: resolveCometIntentRoute(input+workflow_intensity)
IntentRuntime-->>CometClassicCLI: CometIntentRoute{recommendation}
CometClassicCLI-->>User: route + recommendation metadata
User->>CometOpenSkill: start full /comet-open
CometOpenSkill->>CometOpenSkill: [has recommendation]
CometOpenSkill-->>User: present tweak/hotfix/full options
User-->>CometOpenSkill: choose workflow
CometOpenSkill->>CometOpenSkill: run openspec new change (only after confirmation)
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughClassic gains configurable workflow intensity and typed lightweight route recommendations. Routing now returns confirmation metadata for eligible low-risk requests, while risk signals retain full workflow handling. The Open skill adds a blocking recommendation choice before artifact creation in English and Chinese. ChangesLightweight workflow recommendations
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant comet-classic
participant ClassicConfig
participant IntentResolver
participant comet-open
User->>comet-classic: submit intent route
comet-classic->>ClassicConfig: read workflow_intensity
ClassicConfig-->>comet-classic: configured intensity or standard
comet-classic->>IntentResolver: resolve intent with intensity
IntentResolver-->>comet-classic: full route with recommendation and confirmation
comet-classic-->>User: present recommendation options
User->>comet-open: choose tweak, hotfix, or full
comet-open-->>User: continue after explicit confirmation
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey - I've found 2 issues, and left some high level feedback:
- The
WORKFLOW_INTENSITIESconstants are now defined in multiple modules and generated runtimes; consider centralizing the definition in one source module and reusing it to reduce divergence risk between TypeScript and bundled runtime code. - The
lowRiskEvidenceregex currently triggers on generic words likesmalleven when combined with riskier contexts (e.g., "small migration"); if this is too permissive long-term, consider tightening the pattern or adding more field-specific checks so that textual low-risk hints don’t overrule nuanced intent in future extensions.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `WORKFLOW_INTENSITIES` constants are now defined in multiple modules and generated runtimes; consider centralizing the definition in one source module and reusing it to reduce divergence risk between TypeScript and bundled runtime code.
- The `lowRiskEvidence` regex currently triggers on generic words like `small` even when combined with riskier contexts (e.g., "small migration"); if this is too permissive long-term, consider tightening the pattern or adding more field-specific checks so that textual low-risk hints don’t overrule nuanced intent in future extensions.
## Individual Comments
### Comment 1
<location path="domains/comet-classic/classic-intent.ts" line_range="420-424" />
<code_context>
+ return null;
+}
+
function route(
name: CometIntentRouteName,
confidence: number,
fallback_reason: string | null = null,
+ recommendation: CometIntentRecommendation | null = null,
): CometIntentRoute {
const nextSkill: Record<CometIntentRouteName, CometIntentNextSkill | null> = {
</code_context>
<issue_to_address>
**issue (bug_risk):** route no longer returns the constructed CometIntentRoute object
This change removes the `CometIntentRoute` return value, so callers like `resolveCometIntentRoute` now receive `undefined` and will throw when accessing properties such as `resolved.name` or `resolved.recommendation`. Please restore the `return { ... }` for the constructed route object to preserve existing behavior.
</issue_to_address>
### Comment 2
<location path="assets/skills/comet/scripts/comet-runtime.mjs" line_range="12866-12869" />
<code_context>
-function route(name, confidence, fallback_reason = null) {
</code_context>
<issue_to_address>
**issue (bug_risk):** Runtime route helper also drops the return value, mirroring the TS bug
In the compiled runtime `route` helper, the constructed route object is not returned, so calls from `resolveCometIntentRoute` will receive `undefined` and trigger runtime errors when accessing its properties. Please update the runtime implementation to return the constructed route object, consistent with the TypeScript version.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
domains/comet-classic/classic-intent.ts (1)
453-520: 🎯 Functional Correctness | 🟠 Major | 🏗️ Heavy lift
workflow_intensity: 'thorough'can bypass the new confirmation gate via legacy direct-routing branches. Both sites share one root cause: thefix_bug→hotfix andmake_tweak→tweak legacy branches inresolveCometIntentRoutenever checkworkflow_intensity, so whenrecommendationForsuppresses a recommendation under'thorough'+ weak evidence, these older branches still match on evidence alone and route directly tohotfix/tweakwithout confirmation — the opposite of "thorough: recommend the full workflow unless the task is clearly small".
domains/comet-classic/classic-intent.ts#L453-L520: gate thefix_bug/make_tweak/genericworkflow_candidateevidence-based branches onframe.context.workflow_intensity !== 'thorough', falling through to afullroute otherwise; add tests forthorough+ non-smallfix_bug/make_tweak+ workflow_candidate evidence.assets/skills/comet/scripts/comet-runtime.mjs#L12977-L12986: regenerate this bundle from the corrected TS source via the project's build step once the above fix lands; do not patch the generated file directly.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@domains/comet-classic/classic-intent.ts` around lines 453 - 520, Update resolveCometIntentRoute in domains/comet-classic/classic-intent.ts#L453-L520 so the fix_bug→hotfix, make_tweak→tweak, and generic workflow_candidate evidence branches only apply when frame.context.workflow_intensity is not 'thorough'; otherwise fall through to the full-workflow route, and add coverage for thorough non-small fix_bug/make_tweak cases with workflow_candidate evidence. Regenerate assets/skills/comet/scripts/comet-runtime.mjs#L12977-L12986 using the project build step; do not edit the generated bundle directly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@assets/skills/comet-open/SKILL.md`:
- Around line 55-66: Update the Step 1a lightweight recommendation confirmation
in assets/skills/comet-open/SKILL.md lines 55-66 to derive the presented choices
directly from recommendation.options, preserving each option’s existing switch
behavior and requiring the decision pause before artifact creation. Apply the
same synchronized guidance in assets/skills-zh/comet-open/SKILL.md lines 55-66;
do not hardcode or independently infer tweak/hotfix availability.
In `@docs/superpowers/plans/2026-07-29-recommend-lightweight-workflow-paths.md`:
- Line 3: 修正文档中的 design-doc 相对路径：将 `design-doc` 配置值从带有 `comet/` 前缀的路径更新为仓库内实际的
`docs/superpowers/specs/2026-07-29-recommend-lightweight-workflow-paths-design.md`，其余内容保持不变。
In `@test/domains/comet-classic/classic-project-config.test.ts`:
- Around line 69-91: Update the workflow_intensity tests around
readClassicWorkflowIntensity to use one parameterized case covering explicit
light, standard, and thorough values, asserting each value is returned with
source .comet/config.yaml. Keep the separate absent-value default assertion and
invalid-value rejection test, while removing the redundant light-only test.
In `@test/domains/skill/workflow-optimization-contract.test.ts`:
- Around line 56-59: Update the ordering assertions in the workflow optimization
test to also verify that skill.indexOf(guard) is less than
skill.indexOf('openspec new change'). Keep the existing heading presence and
ordering assertions unchanged.
---
Outside diff comments:
In `@domains/comet-classic/classic-intent.ts`:
- Around line 453-520: Update resolveCometIntentRoute in
domains/comet-classic/classic-intent.ts#L453-L520 so the fix_bug→hotfix,
make_tweak→tweak, and generic workflow_candidate evidence branches only apply
when frame.context.workflow_intensity is not 'thorough'; otherwise fall through
to the full-workflow route, and add coverage for thorough non-small
fix_bug/make_tweak cases with workflow_candidate evidence. Regenerate
assets/skills/comet/scripts/comet-runtime.mjs#L12977-L12986 using the project
build step; do not edit the generated bundle directly.
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: e7bbb63f-f4b9-4a96-a75b-c0dd88168c66
📒 Files selected for processing (18)
CHANGELOG.mdREADME-zh.mdREADME.mdassets/skills-zh/comet-open/SKILL.mdassets/skills/comet-native/scripts/comet-native-runtime.mjsassets/skills/comet-open/SKILL.mdassets/skills/comet/scripts/comet-runtime.mjsdocs/superpowers/plans/2026-07-29-recommend-lightweight-workflow-paths.mddocs/superpowers/specs/2026-07-29-recommend-lightweight-workflow-paths-design.mddomains/comet-classic/classic-intent-command.tsdomains/comet-classic/classic-intent.tsdomains/comet-classic/classic-project-config.tsdomains/skill/platform-install.tsdomains/workflow-contract/project-config.tstest/domains/comet-classic/classic-intent.test.tstest/domains/comet-classic/classic-project-config.test.tstest/domains/skill/skills.test.tstest/domains/skill/workflow-optimization-contract.test.ts
googs1025
force-pushed
the
feature/20260729/recommend-lightweight-workflow-paths-pr
branch
from
July 29, 2026 06:56
3f426e2 to
4e47591
Compare
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
test/domains/comet-classic/classic-intent.test.ts (1)
285-285: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win用
as never绕过类型会掩盖真实的 context 类型错误。
workflow_intensity已是CometIntentFrame['context']的合法字段，无需as never。更好的做法是放宽frame()的 overrides 类型（对嵌套对象使用Partial），这样后续字段拼写错误仍能被编译器捕获。♻️ 建议的辅助函数签名调整
-function frame(overrides: Partial<CometIntentFrame> = {}): CometIntentFrame { +type FrameOverrides = Partial<Omit<CometIntentFrame, 'intent' | 'slots' | 'context' | 'proposed_route'>> & { + intent?: Partial<CometIntentFrame['intent']>; + slots?: Partial<CometIntentFrame['slots']>; + context?: Partial<CometIntentFrame['context']>; + proposed_route?: Partial<CometIntentFrame['proposed_route']>; +}; + +function frame(overrides: FrameOverrides = {}): CometIntentFrame {随后即可去掉两处
as never。Also applies to: 309-309
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@test/domains/comet-classic/classic-intent.test.ts` at line 285, 调整测试辅助函数 frame() 的 overrides 类型，使其支持对嵌套 context 使用 Partial<CometIntentFrame['context']>，同时保留字段名校验；随后移除两处 context 对象上的 as never，确保 workflow_intensity 直接按合法字段传入。
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@assets/skills/comet/scripts/comet-runtime.mjs`:
- Around line 13044-13060: 更新 withConfiguredWorkflowIntensity，仅当 input.context
已是 record 时才注入 workflow_intensity；对缺失、数组或字符串等无效 context 直接返回原 input，保留
validateFrame() 的对象校验。对有效 context 继续保留已有 intensity 时不覆盖的行为。
- Around line 12894-12921: Update recommendationFor so every tweak or hotfix
recommendation first requires hasSmallScopeEvidence(frame) to be true. Do not
treat workflow_intensity === "light" as a substitute, and ensure missing or
unknown risk/scope information cannot qualify as low risk; preserve the existing
recommendation branches only after this evidence gate passes.
In `@test/domains/comet-classic/classic-intent.test.ts`:
- Around line 498-549: Update the test around classicIntentCommand to inject
projectRoot through the supported ClassicConfigOptions.cwd path, allowing
readClassicWorkflowIntensity() to locate .comet/config.yaml without calling
process.chdir. Remove previousCwd capture, process.chdir, and restoration while
preserving the temporary directory cleanup and assertions.
---
Nitpick comments:
In `@test/domains/comet-classic/classic-intent.test.ts`:
- Line 285: 调整测试辅助函数 frame() 的 overrides 类型，使其支持对嵌套 context 使用
Partial<CometIntentFrame['context']>，同时保留字段名校验；随后移除两处 context 对象上的 as never，确保
workflow_intensity 直接按合法字段传入。
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 46bd187b-39b5-4d47-afd4-8f4d1edb114f
📒 Files selected for processing (16)
CHANGELOG.mdREADME-zh.mdREADME.mdassets/skills-zh/comet-open/SKILL.mdassets/skills/comet-native/scripts/comet-native-runtime.mjsassets/skills/comet-open/SKILL.mdassets/skills/comet/scripts/comet-runtime.mjsdomains/comet-classic/classic-intent-command.tsdomains/comet-classic/classic-intent.tsdomains/comet-classic/classic-project-config.tsdomains/skill/platform-install.tsdomains/workflow-contract/project-config.tstest/domains/comet-classic/classic-intent.test.tstest/domains/comet-classic/classic-project-config.test.tstest/domains/skill/skills.test.tstest/domains/skill/workflow-optimization-contract.test.ts
🚧 Files skipped from review as they are similar to previous changes (12)
- assets/skills/comet-native/scripts/comet-native-runtime.mjs
- CHANGELOG.md
- domains/comet-classic/classic-intent-command.ts
- domains/workflow-contract/project-config.ts
- domains/skill/platform-install.ts
- README-zh.md
- domains/comet-classic/classic-project-config.ts
- test/domains/skill/workflow-optimization-contract.test.ts
- README.md
- test/domains/skill/skills.test.ts
- domains/comet-classic/classic-intent.ts
- assets/skills-zh/comet-open/SKILL.md
googs1025
changed the title
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@domains/comet-classic/classic-cli.ts`:
- Around line 19-20: Update runClassicCli to accept the new cwd and homeDir
options and forward both alongside json when calling dispatch, allowing
withConfiguredWorkflowIntensity to read configuration from caller-specified
directories. Add coverage verifying custom directory overrides are propagated
and applied.
🪄 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: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: ed1f9546-8203-41ed-837d-a6527bf4b076
📒 Files selected for processing (10)
assets/skills-zh/comet-open/SKILL.mdassets/skills/comet-open/SKILL.mdassets/skills/comet/scripts/comet-runtime.mjsdomains/comet-classic/classic-cli.tsdomains/comet-classic/classic-intent-command.tsdomains/comet-classic/classic-intent.tstest/domains/comet-classic/classic-intent.test.tstest/domains/comet-classic/classic-project-config.test.tstest/domains/comet-classic/classic-runtime.test.tstest/domains/skill/workflow-optimization-contract.test.ts
🚧 Files skipped from review as they are similar to previous changes (7)
- test/domains/skill/workflow-optimization-contract.test.ts
- test/domains/comet-classic/classic-project-config.test.ts
- domains/comet-classic/classic-intent-command.ts
- assets/skills-zh/comet-open/SKILL.md
- test/domains/comet-classic/classic-intent.test.ts
- domains/comet-classic/classic-intent.ts
- assets/skills/comet/scripts/comet-runtime.mjs
Comment on lines
+19
to
+20
| cwd?: string; | ||
| homeDir?: string; |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
转发 cwd 和 homeDir，否则配置注入不会生效。
虽然接口新增了这两个字段，但 runClassicCli 在 Line 119 仍只向 dispatch 传递 { json }。因此下游 withConfiguredWorkflowIntensity 收到的始终是默认选项，无法读取调用方指定目录中的配置，导致 workflow intensity 可能解析错误。请让 runClassicCli 接收并转发这两个字段，并补充自定义目录覆盖测试。
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@domains/comet-classic/classic-cli.ts` around lines 19 - 20, Update
runClassicCli to accept the new cwd and homeDir options and forward both
alongside json when calling dispatch, allowing withConfiguredWorkflowIntensity
to read configuration from caller-specified directories. Add coverage verifying
custom directory overrides are propagated and applied.
Contributor
|
现在这个方案把Agent推断的scope=small当成的推荐的硬性前置条件，但实际上scope是不准的，比如用户可能使用“小改动”等等词开启工作，但实际上又是一个大任务，scope目前也不在意图识别的必填槽位，他本身不太适合有依据的意图 建议调整判定方案，而不是只把 scope 补进意图识别的最小 Frame：
|
googs1025
force-pushed
the
feature/20260729/recommend-lightweight-workflow-paths-pr
branch
from
July 29, 2026 14:53
2906305 to
66d6955
Compare
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
assets/skills/comet-open/SKILL.md (1)
198-198: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win为代码围栏添加语言标识。
两份 Skill 文档的 artifact 目录树代码块都触发 Markdownlint MD040；请将裸代码围栏改为
```text。
assets/skills/comet-open/SKILL.md#L198-L198: 将```改为```text。assets/skills-zh/comet-open/SKILL.md#L198-L198: 将```改为```text。🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@assets/skills/comet-open/SKILL.md` at line 198, 为 artifact 目录树代码块添加语言标识：将 assets/skills/comet-open/SKILL.md 第198行和 assets/skills-zh/comet-open/SKILL.md 第198行的裸代码围栏改为 ```text，保持代码块内容不变。Source: Linters/SAST tools
🧹 Nitpick comments (2)
domains/comet-classic/classic-project-config.ts (2)
30-30: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win补充配置根目录优先级测试。
请覆盖
projectRoot > cwd > invocationCwd > process.cwd()的组合，并验证不同根目录存在配置时读取的是预期文件。当前提供的测试只证明了cwd路径。As per coding guidelines，测试范围必须覆盖当前 TypeScript 改动的最小相关路径。
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@domains/comet-classic/classic-project-config.ts` at line 30, 为 classic-project-config 的配置根目录解析逻辑补充测试，覆盖 projectRoot、cwd、invocationCwd 和 process.cwd() 的优先级组合。为每种根目录分别创建可区分的配置文件，并验证读取结果来自最高优先级的预期目录；同时保留现有仅覆盖 cwd 的场景，确保覆盖当前 TypeScript 改动的最小相关路径。Source: Coding guidelines
17-24: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win统一
workflow_intensity的运行时允许值来源。
classic-project-config.ts和classic-intent.ts各定义了一份相同的WORKFLOW_INTENSITIES；配置读取和 intent 校验都依赖 allowlist，维护两份列表会有漂移风险。放到共用 contract 或导出常量让两端复用同一份。🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@domains/comet-classic/classic-project-config.ts` around lines 17 - 24, 将 workflow_intensity 的允许值集中到一个共享 contract 或导出常量中，移除 classic-project-config.ts 中与 classic-intent.ts 重复的 WORKFLOW_INTENSITIES 定义；更新 ClassicWorkflowIntensity 及相关配置解析逻辑以复用该共享来源，并确保配置读取与 intent 校验继续使用完全相同的运行时 allowlist。
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@assets/skills/comet-open/SKILL.md`:
- Line 198: 为 artifact 目录树代码块添加语言标识：将 assets/skills/comet-open/SKILL.md 第198行和
assets/skills-zh/comet-open/SKILL.md 第198行的裸代码围栏改为 ```text，保持代码块内容不变。
---
Nitpick comments:
In `@domains/comet-classic/classic-project-config.ts`:
- Line 30: 为 classic-project-config 的配置根目录解析逻辑补充测试，覆盖
projectRoot、cwd、invocationCwd 和 process.cwd()
的优先级组合。为每种根目录分别创建可区分的配置文件，并验证读取结果来自最高优先级的预期目录；同时保留现有仅覆盖 cwd 的场景，确保覆盖当前
TypeScript 改动的最小相关路径。
- Around line 17-24: 将 workflow_intensity 的允许值集中到一个共享 contract 或导出常量中，移除
classic-project-config.ts 中与 classic-intent.ts 重复的 WORKFLOW_INTENSITIES 定义；更新
ClassicWorkflowIntensity 及相关配置解析逻辑以复用该共享来源，并确保配置读取与 intent 校验继续使用完全相同的运行时
allowlist。
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 8bd3266f-f92b-4e60-a2b2-7d73a97eda3e
📒 Files selected for processing (13)
CHANGELOG.mdREADME-zh.mdREADME.mdassets/manifest.jsonassets/skills-zh/comet-open/SKILL.mdassets/skills/comet-native/scripts/comet-native-runtime.mjsassets/skills/comet-open/SKILL.mdassets/skills/comet/scripts/comet-runtime.mjsdomains/comet-classic/classic-intent-command.tsdomains/comet-classic/classic-intent.tsdomains/comet-classic/classic-project-config.tsdomains/skill/platform-install.tsdomains/workflow-contract/project-config.ts
🚧 Files skipped from review as they are similar to previous changes (7)
- assets/skills/comet-native/scripts/comet-native-runtime.mjs
- README.md
- README-zh.md
- domains/skill/platform-install.ts
- domains/comet-classic/classic-intent-command.ts
- assets/skills/comet/scripts/comet-runtime.mjs
- domains/comet-classic/classic-intent.ts
Contributor
Author
|
我根据 review comment 和 #213 重新调整了一下 lightweight workflow recommendation 的判断逻辑。 这次主要改动是：不再把
风险判断这块也做了调整。现在会把 另外 我也补了对应测试覆盖，包括：
|
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
2 participants
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
classic.workflow_intensityand typed Classic intent recommendation metadata for lightweight workflow suggestions./comet-opento present tweak/hotfix/full choices before continuing the full workflow.Closes #213
Test Plan
npx vitest run test/domains/comet-classic/classic-intent.test.ts test/domains/comet-classic/classic-project-config.test.ts test/domains/comet-classic/classic-runtime.test.ts test/domains/skill/workflow-optimization-contract.test.ts test/domains/skill/skills.test.tspnpm run check:generatedpnpm lintpnpm buildSummary by CodeRabbit
light,standard, orthorough.tweakorhotfixworkflows for suitable low-risk requests.