omni-context-cli 0.0.81 → 0.0.83
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +4 -2
- package/README.zh-CN.md +4 -2
- package/dist/cli.js +7 -7
- package/dist/clients/extension.vsix +0 -0
- package/dist/skills/simplify/SKILL.md +6 -6
- package/dist/workflows/general.md +7 -11
- package/package.json +1 -1
|
Binary file
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: simplify
|
|
3
|
-
description: Review changed code for reuse, quality, and efficiency, then fix any issues found.
|
|
3
|
+
description: Review changed code for reuse, quality, and efficiency, then fix any issues found.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Simplify: Code Review and Cleanup
|
|
@@ -17,9 +17,9 @@ Run `git diff` (or `git diff HEAD` if there are staged changes) to see what chan
|
|
|
17
17
|
|
|
18
18
|
For each change:
|
|
19
19
|
|
|
20
|
-
1. **Search for existing utilities and helpers** that could replace newly written code. Look for similar patterns elsewhere in the codebase
|
|
20
|
+
1. **Search for existing utilities and helpers** that could replace newly written code. Look for similar patterns elsewhere in the codebase. Common locations are utility directories, shared modules, and files adjacent to the changed ones.
|
|
21
21
|
2. **Flag any new function that duplicates existing functionality.** Suggest the existing function to use instead.
|
|
22
|
-
3. **Flag any inline logic that could use an existing utility
|
|
22
|
+
3. **Flag any inline logic that could use an existing utility**: hand-rolled string manipulation, manual path handling, custom environment checks, ad-hoc type guards, and similar patterns are common candidates.
|
|
23
23
|
|
|
24
24
|
### Code Quality
|
|
25
25
|
|
|
@@ -30,7 +30,7 @@ Review the same changes for hacky patterns:
|
|
|
30
30
|
3. **Copy-paste with slight variation**: near-duplicate code blocks that should be unified with a shared abstraction
|
|
31
31
|
4. **Leaky abstractions**: exposing internal details that should be encapsulated, or breaking existing abstraction boundaries
|
|
32
32
|
5. **Stringly-typed code**: using raw strings where constants, enums (string unions), or branded types already exist in the codebase
|
|
33
|
-
6. **Unnecessary comments**: comments explaining WHAT the code does (well-named identifiers already do that), narrating the change, or referencing the task/caller
|
|
33
|
+
6. **Unnecessary comments**: comments explaining WHAT the code does (well-named identifiers already do that), narrating the change, or referencing the task/caller. Delete them; keep only non-obvious WHY (hidden constraints, subtle invariants, workarounds)
|
|
34
34
|
|
|
35
35
|
### Efficiency
|
|
36
36
|
|
|
@@ -39,8 +39,8 @@ Review the same changes for efficiency:
|
|
|
39
39
|
1. **Unnecessary work**: redundant computations, repeated file reads, duplicate network/API calls, N+1 patterns
|
|
40
40
|
2. **Missed concurrency**: independent operations run sequentially when they could run in parallel
|
|
41
41
|
3. **Hot-path bloat**: new blocking work added to startup or per-request/per-render hot paths
|
|
42
|
-
4. **Recurring no-op updates**: state/store updates inside polling loops, intervals, or event handlers that fire unconditionally
|
|
43
|
-
5. **Unnecessary existence checks**: pre-checking file/resource existence before operating (TOCTOU anti-pattern)
|
|
42
|
+
4. **Recurring no-op updates**: state/store updates inside polling loops, intervals, or event handlers that fire unconditionally. Add a change-detection guard so downstream consumers aren't notified when nothing changed
|
|
43
|
+
5. **Unnecessary existence checks**: pre-checking file/resource existence before operating (TOCTOU anti-pattern). Operate directly and handle the error
|
|
44
44
|
6. **Memory**: unbounded data structures, missing cleanup, event listener leaks
|
|
45
45
|
7. **Overly broad operations**: reading entire files when only a portion is needed, loading all items when filtering for one
|
|
46
46
|
|
|
@@ -6,14 +6,10 @@ allowCustomAgents: true
|
|
|
6
6
|
allowMcpTools: true
|
|
7
7
|
allowRemoteTools: true
|
|
8
8
|
---
|
|
9
|
-
Omx is a versatile personal assistant
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
conversation -- casual when chatting, precise when working on
|
|
17
|
-
something specific. Avoid jargon unless the user uses it first.
|
|
18
|
-
You're running on {{OS_TYPE}} ({{PLATFORM}}/{{ARCH}}). The current
|
|
19
|
-
working directory is {{CWD}}. Today is {{TODAY}}.
|
|
9
|
+
Omx is a versatile personal assistant. Keep your responses clear,
|
|
10
|
+
friendly, and to the point. Match the tone of the conversation. Be
|
|
11
|
+
casual when chatting, precise when working on something specific.
|
|
12
|
+
Avoid jargon unless the user uses it first. When you have tools
|
|
13
|
+
available, use them proactively rather than guessing. You're running
|
|
14
|
+
on {{OS_TYPE}} ({{PLATFORM}}/{{ARCH}}). The current working directory
|
|
15
|
+
is {{CWD}}. Today is {{TODAY}}.
|