prizmkit 1.0.12 → 1.0.14
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/bin/create-prizmkit.js +4 -1
- package/bundled/VERSION.json +3 -3
- package/bundled/adapters/claude/command-adapter.js +35 -4
- package/bundled/adapters/claude/rules-adapter.js +6 -58
- package/bundled/adapters/claude/team-adapter.js +2 -2
- package/bundled/adapters/codebuddy/agent-adapter.js +0 -1
- package/bundled/adapters/codebuddy/rules-adapter.js +30 -0
- package/bundled/adapters/shared/frontmatter.js +3 -1
- package/bundled/dev-pipeline/README.md +13 -3
- package/bundled/dev-pipeline/launch-bugfix-daemon.sh +10 -0
- package/bundled/dev-pipeline/launch-daemon.sh +18 -4
- package/bundled/dev-pipeline/lib/common.sh +105 -0
- package/bundled/dev-pipeline/retry-bug.sh +12 -0
- package/bundled/dev-pipeline/retry-feature.sh +12 -0
- package/bundled/dev-pipeline/run-bugfix.sh +71 -57
- package/bundled/dev-pipeline/run.sh +87 -57
- package/bundled/dev-pipeline/scripts/check-session-status.py +47 -2
- package/bundled/dev-pipeline/scripts/cleanup-logs.py +192 -0
- package/bundled/dev-pipeline/scripts/detect-stuck.py +15 -3
- package/bundled/dev-pipeline/scripts/generate-bootstrap-prompt.py +32 -27
- package/bundled/dev-pipeline/scripts/generate-bugfix-prompt.py +23 -23
- package/bundled/dev-pipeline/scripts/update-feature-status.py +73 -2
- package/bundled/dev-pipeline/scripts/utils.py +22 -0
- package/bundled/dev-pipeline/templates/bootstrap-tier1.md +38 -2
- package/bundled/dev-pipeline/templates/bootstrap-tier2.md +39 -2
- package/bundled/dev-pipeline/templates/bootstrap-tier3.md +39 -2
- package/bundled/dev-pipeline/templates/session-status-schema.json +7 -1
- package/bundled/dev-pipeline/tests/__init__.py +0 -0
- package/bundled/dev-pipeline/tests/conftest.py +133 -0
- package/bundled/dev-pipeline/tests/test_check_session.py +127 -0
- package/bundled/dev-pipeline/tests/test_cleanup_logs.py +119 -0
- package/bundled/dev-pipeline/tests/test_detect_stuck.py +207 -0
- package/bundled/dev-pipeline/tests/test_generate_bugfix_prompt.py +181 -0
- package/bundled/dev-pipeline/tests/test_generate_prompt.py +190 -0
- package/bundled/dev-pipeline/tests/test_init_bugfix_pipeline.py +153 -0
- package/bundled/dev-pipeline/tests/test_init_pipeline.py +241 -0
- package/bundled/dev-pipeline/tests/test_update_bug_status.py +142 -0
- package/bundled/dev-pipeline/tests/test_update_feature_status.py +277 -0
- package/bundled/dev-pipeline/tests/test_utils.py +141 -0
- package/bundled/rules/USAGE.md +153 -0
- package/bundled/rules/_rules-metadata.json +43 -0
- package/bundled/rules/general/prefer-linux-commands.md +9 -0
- package/bundled/rules/prizm/prizm-commit-workflow.md +10 -0
- package/bundled/rules/prizm/prizm-documentation.md +19 -0
- package/bundled/rules/prizm/prizm-progressive-loading.md +11 -0
- package/bundled/skills/_metadata.json +130 -67
- package/bundled/skills/app-planner/SKILL.md +252 -499
- package/bundled/skills/app-planner/assets/evaluation-guide.md +44 -0
- package/bundled/skills/app-planner/scripts/validate-and-generate.py +143 -4
- package/bundled/skills/bug-planner/SKILL.md +58 -13
- package/bundled/skills/bugfix-pipeline-launcher/SKILL.md +5 -7
- package/bundled/skills/dev-pipeline-launcher/SKILL.md +16 -7
- package/bundled/skills/feature-workflow/SKILL.md +175 -234
- package/bundled/skills/prizm-kit/SKILL.md +17 -31
- package/bundled/skills/{prizmkit-adr-manager → prizmkit-tool-adr-manager}/SKILL.md +6 -7
- package/bundled/skills/{prizmkit-api-doc-generator → prizmkit-tool-api-doc-generator}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-bug-reproducer → prizmkit-tool-bug-reproducer}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-ci-cd-generator → prizmkit-tool-ci-cd-generator}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-db-migration → prizmkit-tool-db-migration}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-dependency-health → prizmkit-tool-dependency-health}/SKILL.md +3 -4
- package/bundled/skills/{prizmkit-deployment-strategy → prizmkit-tool-deployment-strategy}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-error-triage → prizmkit-tool-error-triage}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-log-analyzer → prizmkit-tool-log-analyzer}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-monitoring-setup → prizmkit-tool-monitoring-setup}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-onboarding-generator → prizmkit-tool-onboarding-generator}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-perf-profiler → prizmkit-tool-perf-profiler}/SKILL.md +4 -5
- package/bundled/skills/{prizmkit-security-audit → prizmkit-tool-security-audit}/SKILL.md +3 -4
- package/bundled/skills/{prizmkit-tech-debt-tracker → prizmkit-tool-tech-debt-tracker}/SKILL.md +3 -4
- package/bundled/skills/refactor-skill/SKILL.md +371 -0
- package/bundled/skills/refactor-workflow/SKILL.md +17 -119
- package/package.json +1 -1
- package/src/external-skills.js +71 -0
- package/src/index.js +62 -4
- package/src/metadata.js +36 -0
- package/src/scaffold.js +136 -32
- package/bundled/skills/prizmkit-bug-fix-workflow/SKILL.md +0 -356
- package/bundled/templates/claude-md-template.md +0 -38
- package/bundled/templates/codebuddy-md-template.md +0 -35
- /package/bundled/skills/{prizmkit-adr-manager → prizmkit-tool-adr-manager}/assets/adr-template.md +0 -0
|
@@ -0,0 +1,207 @@
|
|
|
1
|
+
"""Tests for detect-stuck.py."""
|
|
2
|
+
|
|
3
|
+
import json
|
|
4
|
+
import os
|
|
5
|
+
import sys
|
|
6
|
+
import pytest
|
|
7
|
+
from datetime import datetime, timezone, timedelta
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
def _import_detect_stuck():
|
|
11
|
+
import importlib.util
|
|
12
|
+
path = os.path.join(
|
|
13
|
+
os.path.dirname(__file__), "..", "scripts", "detect-stuck.py"
|
|
14
|
+
)
|
|
15
|
+
spec = importlib.util.spec_from_file_location("detect_stuck", path)
|
|
16
|
+
mod = importlib.util.module_from_spec(spec)
|
|
17
|
+
sys.modules["detect_stuck"] = mod
|
|
18
|
+
spec.loader.exec_module(mod)
|
|
19
|
+
return mod
|
|
20
|
+
|
|
21
|
+
|
|
22
|
+
ds = _import_detect_stuck()
|
|
23
|
+
parse_iso_timestamp = ds.parse_iso_timestamp
|
|
24
|
+
check_max_retries = ds.check_max_retries
|
|
25
|
+
check_stuck_checkpoint = ds.check_stuck_checkpoint
|
|
26
|
+
check_dependency_deadlock = ds.check_dependency_deadlock
|
|
27
|
+
|
|
28
|
+
|
|
29
|
+
class TestParseIsoTimestamp:
|
|
30
|
+
def test_standard_z_format(self):
|
|
31
|
+
result = parse_iso_timestamp("2024-01-15T10:30:00Z")
|
|
32
|
+
assert result is not None
|
|
33
|
+
assert result.year == 2024
|
|
34
|
+
assert result.month == 1
|
|
35
|
+
assert result.day == 15
|
|
36
|
+
assert result.hour == 10
|
|
37
|
+
assert result.minute == 30
|
|
38
|
+
|
|
39
|
+
def test_plus_zero_offset(self):
|
|
40
|
+
result = parse_iso_timestamp("2024-01-15T10:30:00+00:00")
|
|
41
|
+
assert result is not None
|
|
42
|
+
assert result.year == 2024
|
|
43
|
+
|
|
44
|
+
def test_with_microseconds_z(self):
|
|
45
|
+
result = parse_iso_timestamp("2024-01-15T10:30:00.123456Z")
|
|
46
|
+
assert result is not None
|
|
47
|
+
|
|
48
|
+
def test_with_microseconds_offset(self):
|
|
49
|
+
result = parse_iso_timestamp("2024-01-15T10:30:00.123456+00:00")
|
|
50
|
+
assert result is not None
|
|
51
|
+
|
|
52
|
+
def test_invalid_string(self):
|
|
53
|
+
result = parse_iso_timestamp("not a timestamp")
|
|
54
|
+
assert result is None
|
|
55
|
+
|
|
56
|
+
def test_none_input(self):
|
|
57
|
+
result = parse_iso_timestamp(None)
|
|
58
|
+
assert result is None
|
|
59
|
+
|
|
60
|
+
def test_integer_input(self):
|
|
61
|
+
result = parse_iso_timestamp(12345)
|
|
62
|
+
assert result is None
|
|
63
|
+
|
|
64
|
+
def test_empty_string(self):
|
|
65
|
+
result = parse_iso_timestamp("")
|
|
66
|
+
assert result is None
|
|
67
|
+
|
|
68
|
+
|
|
69
|
+
class TestCheckMaxRetries:
|
|
70
|
+
def test_below_max(self):
|
|
71
|
+
status = {"retry_count": 1}
|
|
72
|
+
result = check_max_retries(status, 3)
|
|
73
|
+
assert result is None
|
|
74
|
+
|
|
75
|
+
def test_at_max(self):
|
|
76
|
+
status = {"retry_count": 3}
|
|
77
|
+
result = check_max_retries(status, 3)
|
|
78
|
+
assert result is not None
|
|
79
|
+
assert result["reason"] == "max_retries_exceeded"
|
|
80
|
+
|
|
81
|
+
def test_above_max(self):
|
|
82
|
+
status = {"retry_count": 5}
|
|
83
|
+
result = check_max_retries(status, 3)
|
|
84
|
+
assert result is not None
|
|
85
|
+
|
|
86
|
+
def test_zero_retries(self):
|
|
87
|
+
status = {"retry_count": 0}
|
|
88
|
+
result = check_max_retries(status, 3)
|
|
89
|
+
assert result is None
|
|
90
|
+
|
|
91
|
+
def test_missing_retry_count(self):
|
|
92
|
+
status = {}
|
|
93
|
+
result = check_max_retries(status, 3)
|
|
94
|
+
assert result is None # defaults to 0
|
|
95
|
+
|
|
96
|
+
def test_non_int_retry_count(self):
|
|
97
|
+
status = {"retry_count": "not a number"}
|
|
98
|
+
result = check_max_retries(status, 3)
|
|
99
|
+
assert result is None
|
|
100
|
+
|
|
101
|
+
|
|
102
|
+
class TestCheckStuckCheckpoint:
|
|
103
|
+
def _setup_sessions(self, tmp_path, checkpoints):
|
|
104
|
+
"""Create a feature dir with sessions having given checkpoints."""
|
|
105
|
+
feature_dir = tmp_path / "feature"
|
|
106
|
+
sessions_dir = feature_dir / "sessions"
|
|
107
|
+
for i, cp in enumerate(checkpoints):
|
|
108
|
+
session_dir = sessions_dir / "session-{:03d}".format(i)
|
|
109
|
+
session_dir.mkdir(parents=True)
|
|
110
|
+
data = {"checkpoint_reached": cp}
|
|
111
|
+
(session_dir / "session-status.json").write_text(
|
|
112
|
+
json.dumps(data), encoding="utf-8"
|
|
113
|
+
)
|
|
114
|
+
return str(feature_dir)
|
|
115
|
+
|
|
116
|
+
def test_stuck_at_same_checkpoint(self, tmp_path):
|
|
117
|
+
feature_dir = self._setup_sessions(tmp_path, ["CP-3", "CP-3", "CP-3"])
|
|
118
|
+
result = check_stuck_checkpoint(feature_dir)
|
|
119
|
+
assert result is not None
|
|
120
|
+
assert result["reason"] == "stuck_at_checkpoint"
|
|
121
|
+
assert "CP-3" in result["details"]
|
|
122
|
+
|
|
123
|
+
def test_different_checkpoints(self, tmp_path):
|
|
124
|
+
feature_dir = self._setup_sessions(tmp_path, ["CP-1", "CP-2", "CP-3"])
|
|
125
|
+
result = check_stuck_checkpoint(feature_dir)
|
|
126
|
+
assert result is None
|
|
127
|
+
|
|
128
|
+
def test_fewer_than_three_sessions(self, tmp_path):
|
|
129
|
+
feature_dir = self._setup_sessions(tmp_path, ["CP-3", "CP-3"])
|
|
130
|
+
result = check_stuck_checkpoint(feature_dir)
|
|
131
|
+
assert result is None
|
|
132
|
+
|
|
133
|
+
def test_no_sessions(self, tmp_path):
|
|
134
|
+
feature_dir = tmp_path / "feature"
|
|
135
|
+
feature_dir.mkdir()
|
|
136
|
+
result = check_stuck_checkpoint(str(feature_dir))
|
|
137
|
+
assert result is None
|
|
138
|
+
|
|
139
|
+
def test_none_checkpoints(self, tmp_path):
|
|
140
|
+
feature_dir = self._setup_sessions(tmp_path, [None, None, None])
|
|
141
|
+
result = check_stuck_checkpoint(feature_dir)
|
|
142
|
+
assert result is None # None checkpoints don't trigger stuck
|
|
143
|
+
|
|
144
|
+
def test_mixed_with_last_three_same(self, tmp_path):
|
|
145
|
+
feature_dir = self._setup_sessions(
|
|
146
|
+
tmp_path, ["CP-1", "CP-2", "CP-3", "CP-3", "CP-3"]
|
|
147
|
+
)
|
|
148
|
+
result = check_stuck_checkpoint(feature_dir)
|
|
149
|
+
assert result is not None
|
|
150
|
+
assert "CP-3" in result["details"]
|
|
151
|
+
|
|
152
|
+
|
|
153
|
+
class TestCheckDependencyDeadlock:
|
|
154
|
+
def test_dependency_failed(self, state_dir):
|
|
155
|
+
# Create a failed dependency
|
|
156
|
+
dep_dir = os.path.join(state_dir, "features", "F-001")
|
|
157
|
+
os.makedirs(dep_dir, exist_ok=True)
|
|
158
|
+
with open(os.path.join(dep_dir, "status.json"), "w") as f:
|
|
159
|
+
json.dump({"status": "failed"}, f)
|
|
160
|
+
|
|
161
|
+
feature_list_data = {
|
|
162
|
+
"features": [
|
|
163
|
+
{"id": "F-001", "dependencies": []},
|
|
164
|
+
{"id": "F-002", "dependencies": ["F-001"]},
|
|
165
|
+
]
|
|
166
|
+
}
|
|
167
|
+
result = check_dependency_deadlock("F-002", feature_list_data, state_dir)
|
|
168
|
+
assert result is not None
|
|
169
|
+
assert result["reason"] == "dependency_failed"
|
|
170
|
+
assert "F-001" in result["details"]
|
|
171
|
+
|
|
172
|
+
def test_dependency_completed(self, state_dir):
|
|
173
|
+
dep_dir = os.path.join(state_dir, "features", "F-001")
|
|
174
|
+
os.makedirs(dep_dir, exist_ok=True)
|
|
175
|
+
with open(os.path.join(dep_dir, "status.json"), "w") as f:
|
|
176
|
+
json.dump({"status": "completed"}, f)
|
|
177
|
+
|
|
178
|
+
feature_list_data = {
|
|
179
|
+
"features": [
|
|
180
|
+
{"id": "F-001", "dependencies": []},
|
|
181
|
+
{"id": "F-002", "dependencies": ["F-001"]},
|
|
182
|
+
]
|
|
183
|
+
}
|
|
184
|
+
result = check_dependency_deadlock("F-002", feature_list_data, state_dir)
|
|
185
|
+
assert result is None
|
|
186
|
+
|
|
187
|
+
def test_no_dependencies(self, state_dir):
|
|
188
|
+
feature_list_data = {
|
|
189
|
+
"features": [
|
|
190
|
+
{"id": "F-001", "dependencies": []},
|
|
191
|
+
]
|
|
192
|
+
}
|
|
193
|
+
result = check_dependency_deadlock("F-001", feature_list_data, state_dir)
|
|
194
|
+
assert result is None
|
|
195
|
+
|
|
196
|
+
def test_none_feature_list(self, state_dir):
|
|
197
|
+
result = check_dependency_deadlock("F-001", None, state_dir)
|
|
198
|
+
assert result is None
|
|
199
|
+
|
|
200
|
+
def test_feature_not_found(self, state_dir):
|
|
201
|
+
feature_list_data = {
|
|
202
|
+
"features": [
|
|
203
|
+
{"id": "F-001", "dependencies": []},
|
|
204
|
+
]
|
|
205
|
+
}
|
|
206
|
+
result = check_dependency_deadlock("F-999", feature_list_data, state_dir)
|
|
207
|
+
assert result is None
|
|
@@ -0,0 +1,181 @@
|
|
|
1
|
+
"""Tests for generate-bugfix-prompt.py."""
|
|
2
|
+
|
|
3
|
+
import os
|
|
4
|
+
import sys
|
|
5
|
+
import pytest
|
|
6
|
+
|
|
7
|
+
|
|
8
|
+
def _import_generate_bugfix_prompt():
|
|
9
|
+
import importlib.util
|
|
10
|
+
path = os.path.join(
|
|
11
|
+
os.path.dirname(__file__), "..", "scripts", "generate-bugfix-prompt.py"
|
|
12
|
+
)
|
|
13
|
+
spec = importlib.util.spec_from_file_location("generate_bugfix_prompt", path)
|
|
14
|
+
mod = importlib.util.module_from_spec(spec)
|
|
15
|
+
sys.modules["generate_bugfix_prompt"] = mod
|
|
16
|
+
spec.loader.exec_module(mod)
|
|
17
|
+
return mod
|
|
18
|
+
|
|
19
|
+
|
|
20
|
+
gen_bugfix = _import_generate_bugfix_prompt()
|
|
21
|
+
format_acceptance_criteria = gen_bugfix.format_acceptance_criteria
|
|
22
|
+
format_global_context = gen_bugfix.format_global_context
|
|
23
|
+
format_error_source_details = gen_bugfix.format_error_source_details
|
|
24
|
+
format_environment = gen_bugfix.format_environment
|
|
25
|
+
process_conditional_blocks = gen_bugfix.process_conditional_blocks
|
|
26
|
+
|
|
27
|
+
|
|
28
|
+
class TestFormatAcceptanceCriteria:
|
|
29
|
+
def test_normal(self):
|
|
30
|
+
result = format_acceptance_criteria(["Fix A", "Fix B"])
|
|
31
|
+
assert result == "- Fix A\n- Fix B"
|
|
32
|
+
|
|
33
|
+
def test_empty(self):
|
|
34
|
+
result = format_acceptance_criteria([])
|
|
35
|
+
assert "none specified" in result
|
|
36
|
+
|
|
37
|
+
def test_none(self):
|
|
38
|
+
result = format_acceptance_criteria(None)
|
|
39
|
+
assert "none specified" in result
|
|
40
|
+
|
|
41
|
+
def test_single(self):
|
|
42
|
+
result = format_acceptance_criteria(["Only one"])
|
|
43
|
+
assert result == "- Only one"
|
|
44
|
+
|
|
45
|
+
|
|
46
|
+
class TestFormatGlobalContext:
|
|
47
|
+
def test_normal(self):
|
|
48
|
+
result = format_global_context({"lang": "Python"})
|
|
49
|
+
assert "**lang**" in result
|
|
50
|
+
assert "Python" in result
|
|
51
|
+
|
|
52
|
+
def test_empty(self):
|
|
53
|
+
assert "none specified" in format_global_context({})
|
|
54
|
+
|
|
55
|
+
def test_none(self):
|
|
56
|
+
assert "none specified" in format_global_context(None)
|
|
57
|
+
|
|
58
|
+
|
|
59
|
+
class TestFormatErrorSourceDetails:
|
|
60
|
+
def test_stack_trace(self):
|
|
61
|
+
error_source = {
|
|
62
|
+
"type": "stack_trace",
|
|
63
|
+
"stack_trace": "Traceback ... Error",
|
|
64
|
+
"error_message": "Something broke",
|
|
65
|
+
}
|
|
66
|
+
result = format_error_source_details(error_source)
|
|
67
|
+
assert "Stack Trace" in result
|
|
68
|
+
assert "Traceback ... Error" in result
|
|
69
|
+
assert "Error Message" in result
|
|
70
|
+
assert "Something broke" in result
|
|
71
|
+
|
|
72
|
+
def test_log_pattern(self):
|
|
73
|
+
error_source = {
|
|
74
|
+
"type": "log_pattern",
|
|
75
|
+
"log_snippet": "ERROR 2024-01-01 ...",
|
|
76
|
+
}
|
|
77
|
+
result = format_error_source_details(error_source)
|
|
78
|
+
assert "Log Snippet" in result
|
|
79
|
+
|
|
80
|
+
def test_failed_test(self):
|
|
81
|
+
error_source = {
|
|
82
|
+
"type": "failed_test",
|
|
83
|
+
"failed_test_path": "tests/test_auth.py::test_login",
|
|
84
|
+
}
|
|
85
|
+
result = format_error_source_details(error_source)
|
|
86
|
+
assert "Failed Test" in result
|
|
87
|
+
assert "test_auth.py" in result
|
|
88
|
+
|
|
89
|
+
def test_user_report(self):
|
|
90
|
+
error_source = {
|
|
91
|
+
"type": "user_report",
|
|
92
|
+
"reproduction_steps": ["Step 1", "Step 2"],
|
|
93
|
+
}
|
|
94
|
+
result = format_error_source_details(error_source)
|
|
95
|
+
assert "Reproduction Steps" in result
|
|
96
|
+
assert "Step 1" in result
|
|
97
|
+
assert "Step 2" in result
|
|
98
|
+
|
|
99
|
+
def test_none_source(self):
|
|
100
|
+
result = format_error_source_details(None)
|
|
101
|
+
assert "no error source details" in result
|
|
102
|
+
|
|
103
|
+
def test_empty_source(self):
|
|
104
|
+
result = format_error_source_details({})
|
|
105
|
+
assert "no error source details" in result
|
|
106
|
+
|
|
107
|
+
def test_non_dict_source(self):
|
|
108
|
+
result = format_error_source_details("not a dict")
|
|
109
|
+
assert "no error source details" in result
|
|
110
|
+
|
|
111
|
+
def test_unknown_type_with_error_message(self):
|
|
112
|
+
error_source = {
|
|
113
|
+
"type": "unknown",
|
|
114
|
+
"error_message": "Something happened",
|
|
115
|
+
}
|
|
116
|
+
result = format_error_source_details(error_source)
|
|
117
|
+
assert "Error Message" in result
|
|
118
|
+
|
|
119
|
+
|
|
120
|
+
class TestFormatEnvironment:
|
|
121
|
+
def test_normal(self):
|
|
122
|
+
result = format_environment({"os": "macOS", "browser": "Chrome"})
|
|
123
|
+
assert "**browser**" in result
|
|
124
|
+
assert "**os**" in result
|
|
125
|
+
|
|
126
|
+
def test_empty(self):
|
|
127
|
+
result = format_environment({})
|
|
128
|
+
assert "not specified" in result
|
|
129
|
+
|
|
130
|
+
def test_none(self):
|
|
131
|
+
result = format_environment(None)
|
|
132
|
+
assert "not specified" in result
|
|
133
|
+
|
|
134
|
+
def test_non_dict(self):
|
|
135
|
+
result = format_environment("not a dict")
|
|
136
|
+
assert "not specified" in result
|
|
137
|
+
|
|
138
|
+
def test_empty_values_filtered(self):
|
|
139
|
+
result = format_environment({"os": "macOS", "empty_key": ""})
|
|
140
|
+
assert "os" in result
|
|
141
|
+
# empty_key should be filtered because its value is falsy
|
|
142
|
+
assert "empty_key" not in result
|
|
143
|
+
|
|
144
|
+
def test_all_empty_values(self):
|
|
145
|
+
result = format_environment({"a": "", "b": None})
|
|
146
|
+
assert "not specified" in result
|
|
147
|
+
|
|
148
|
+
|
|
149
|
+
class TestProcessConditionalBlocks:
|
|
150
|
+
def test_manual_verification_keeps_block(self):
|
|
151
|
+
bug = {"verification_type": "manual"}
|
|
152
|
+
content = "before\n{{IF_VERIFICATION_MANUAL_OR_HYBRID}}\nmanual stuff\n{{END_IF_VERIFICATION_MANUAL_OR_HYBRID}}\nafter"
|
|
153
|
+
result = process_conditional_blocks(content, bug)
|
|
154
|
+
assert "manual stuff" in result
|
|
155
|
+
assert "{{IF_VERIFICATION" not in result
|
|
156
|
+
|
|
157
|
+
def test_hybrid_verification_keeps_block(self):
|
|
158
|
+
bug = {"verification_type": "hybrid"}
|
|
159
|
+
content = "{{IF_VERIFICATION_MANUAL_OR_HYBRID}}\nhybrid stuff\n{{END_IF_VERIFICATION_MANUAL_OR_HYBRID}}"
|
|
160
|
+
result = process_conditional_blocks(content, bug)
|
|
161
|
+
assert "hybrid stuff" in result
|
|
162
|
+
|
|
163
|
+
def test_automated_verification_removes_block(self):
|
|
164
|
+
bug = {"verification_type": "automated"}
|
|
165
|
+
content = "before\n{{IF_VERIFICATION_MANUAL_OR_HYBRID}}\nmanual stuff\n{{END_IF_VERIFICATION_MANUAL_OR_HYBRID}}\nafter"
|
|
166
|
+
result = process_conditional_blocks(content, bug)
|
|
167
|
+
assert "manual stuff" not in result
|
|
168
|
+
assert "after" in result
|
|
169
|
+
|
|
170
|
+
def test_no_verification_type_defaults_automated(self):
|
|
171
|
+
bug = {}
|
|
172
|
+
content = "{{IF_VERIFICATION_MANUAL_OR_HYBRID}}\nstuff\n{{END_IF_VERIFICATION_MANUAL_OR_HYBRID}}\nrest"
|
|
173
|
+
result = process_conditional_blocks(content, bug)
|
|
174
|
+
assert "stuff" not in result
|
|
175
|
+
assert "rest" in result
|
|
176
|
+
|
|
177
|
+
def test_no_conditional_blocks(self):
|
|
178
|
+
bug = {"verification_type": "automated"}
|
|
179
|
+
content = "plain text"
|
|
180
|
+
result = process_conditional_blocks(content, bug)
|
|
181
|
+
assert result == "plain text"
|
|
@@ -0,0 +1,190 @@
|
|
|
1
|
+
"""Tests for generate-bootstrap-prompt.py."""
|
|
2
|
+
|
|
3
|
+
import os
|
|
4
|
+
import sys
|
|
5
|
+
import pytest
|
|
6
|
+
|
|
7
|
+
|
|
8
|
+
def _import_generate_prompt():
|
|
9
|
+
import importlib.util
|
|
10
|
+
path = os.path.join(
|
|
11
|
+
os.path.dirname(__file__), "..", "scripts", "generate-bootstrap-prompt.py"
|
|
12
|
+
)
|
|
13
|
+
spec = importlib.util.spec_from_file_location("generate_bootstrap_prompt", path)
|
|
14
|
+
mod = importlib.util.module_from_spec(spec)
|
|
15
|
+
sys.modules["generate_bootstrap_prompt"] = mod
|
|
16
|
+
spec.loader.exec_module(mod)
|
|
17
|
+
return mod
|
|
18
|
+
|
|
19
|
+
|
|
20
|
+
gen_prompt = _import_generate_prompt()
|
|
21
|
+
compute_feature_slug = gen_prompt.compute_feature_slug
|
|
22
|
+
format_acceptance_criteria = gen_prompt.format_acceptance_criteria
|
|
23
|
+
format_global_context = gen_prompt.format_global_context
|
|
24
|
+
process_conditional_blocks = gen_prompt.process_conditional_blocks
|
|
25
|
+
process_mode_blocks = gen_prompt.process_mode_blocks
|
|
26
|
+
determine_pipeline_mode = gen_prompt.determine_pipeline_mode
|
|
27
|
+
|
|
28
|
+
|
|
29
|
+
class TestComputeFeatureSlug:
|
|
30
|
+
def test_basic(self):
|
|
31
|
+
result = compute_feature_slug("F-001", "Project Infrastructure Setup")
|
|
32
|
+
assert result == "001-project-infrastructure-setup"
|
|
33
|
+
|
|
34
|
+
def test_special_characters(self):
|
|
35
|
+
result = compute_feature_slug("F-002", "User Auth (OAuth2.0)")
|
|
36
|
+
assert result == "002-user-auth-oauth20"
|
|
37
|
+
|
|
38
|
+
def test_uppercase_id(self):
|
|
39
|
+
result = compute_feature_slug("F-042", "My Feature")
|
|
40
|
+
assert result == "042-my-feature"
|
|
41
|
+
|
|
42
|
+
def test_lowercase_f(self):
|
|
43
|
+
result = compute_feature_slug("f-5", "Quick Fix")
|
|
44
|
+
assert result == "005-quick-fix"
|
|
45
|
+
|
|
46
|
+
def test_extra_spaces(self):
|
|
47
|
+
result = compute_feature_slug("F-010", " Extra Spaces ")
|
|
48
|
+
assert result == "010-extra-spaces"
|
|
49
|
+
|
|
50
|
+
def test_empty_title(self):
|
|
51
|
+
result = compute_feature_slug("F-001", "")
|
|
52
|
+
# Numeric part only, no trailing hyphen
|
|
53
|
+
assert result.startswith("001")
|
|
54
|
+
|
|
55
|
+
def test_numeric_padding(self):
|
|
56
|
+
result = compute_feature_slug("F-1", "Test")
|
|
57
|
+
assert result.startswith("001-")
|
|
58
|
+
|
|
59
|
+
|
|
60
|
+
class TestFormatAcceptanceCriteria:
|
|
61
|
+
def test_single_item(self):
|
|
62
|
+
result = format_acceptance_criteria(["Must work"])
|
|
63
|
+
assert result == "- Must work"
|
|
64
|
+
|
|
65
|
+
def test_multiple_items(self):
|
|
66
|
+
result = format_acceptance_criteria(["A", "B", "C"])
|
|
67
|
+
assert result == "- A\n- B\n- C"
|
|
68
|
+
|
|
69
|
+
def test_empty_list(self):
|
|
70
|
+
result = format_acceptance_criteria([])
|
|
71
|
+
assert "none specified" in result
|
|
72
|
+
|
|
73
|
+
def test_none_input(self):
|
|
74
|
+
result = format_acceptance_criteria(None)
|
|
75
|
+
assert "none specified" in result
|
|
76
|
+
|
|
77
|
+
|
|
78
|
+
class TestFormatGlobalContext:
|
|
79
|
+
def test_basic(self):
|
|
80
|
+
result = format_global_context({"language": "Python", "framework": "FastAPI"})
|
|
81
|
+
assert "**framework**" in result
|
|
82
|
+
assert "**language**" in result
|
|
83
|
+
assert "Python" in result
|
|
84
|
+
assert "FastAPI" in result
|
|
85
|
+
|
|
86
|
+
def test_sorted_keys(self):
|
|
87
|
+
result = format_global_context({"z_key": "z", "a_key": "a"})
|
|
88
|
+
lines = result.split("\n")
|
|
89
|
+
assert "a_key" in lines[0]
|
|
90
|
+
assert "z_key" in lines[1]
|
|
91
|
+
|
|
92
|
+
def test_empty_context(self):
|
|
93
|
+
result = format_global_context({})
|
|
94
|
+
assert "none specified" in result
|
|
95
|
+
|
|
96
|
+
def test_none_context(self):
|
|
97
|
+
result = format_global_context(None)
|
|
98
|
+
assert "none specified" in result
|
|
99
|
+
|
|
100
|
+
|
|
101
|
+
class TestProcessConditionalBlocks:
|
|
102
|
+
def test_fresh_start_keeps_fresh_block(self):
|
|
103
|
+
content = "before\n{{IF_FRESH_START}}\nfresh content\n{{END_IF_FRESH_START}}\nafter"
|
|
104
|
+
result = process_conditional_blocks(content, "null")
|
|
105
|
+
assert "fresh content" in result
|
|
106
|
+
assert "{{IF_FRESH_START}}" not in result
|
|
107
|
+
assert "after" in result
|
|
108
|
+
|
|
109
|
+
def test_fresh_start_removes_resume_block(self):
|
|
110
|
+
content = "before\n{{IF_RESUME}}\nresume content\n{{END_IF_RESUME}}\nafter"
|
|
111
|
+
result = process_conditional_blocks(content, "null")
|
|
112
|
+
assert "resume content" not in result
|
|
113
|
+
assert "after" in result
|
|
114
|
+
|
|
115
|
+
def test_resume_keeps_resume_block(self):
|
|
116
|
+
content = "before\n{{IF_RESUME}}\nresume content\n{{END_IF_RESUME}}\nafter"
|
|
117
|
+
result = process_conditional_blocks(content, "3")
|
|
118
|
+
assert "resume content" in result
|
|
119
|
+
assert "{{IF_RESUME}}" not in result
|
|
120
|
+
|
|
121
|
+
def test_resume_removes_fresh_block(self):
|
|
122
|
+
content = "before\n{{IF_FRESH_START}}\nfresh content\n{{END_IF_FRESH_START}}\nafter"
|
|
123
|
+
result = process_conditional_blocks(content, "3")
|
|
124
|
+
assert "fresh content" not in result
|
|
125
|
+
assert "after" in result
|
|
126
|
+
|
|
127
|
+
def test_no_conditional_blocks(self):
|
|
128
|
+
content = "just plain text"
|
|
129
|
+
result = process_conditional_blocks(content, "null")
|
|
130
|
+
assert result == "just plain text"
|
|
131
|
+
|
|
132
|
+
|
|
133
|
+
class TestProcessModeBlocks:
|
|
134
|
+
def test_keeps_matching_mode(self):
|
|
135
|
+
content = "before\n{{IF_MODE_LITE}}\nlite content\n{{END_IF_MODE_LITE}}\nafter"
|
|
136
|
+
result = process_mode_blocks(content, "lite", False)
|
|
137
|
+
assert "lite content" in result
|
|
138
|
+
assert "{{IF_MODE_LITE}}" not in result
|
|
139
|
+
|
|
140
|
+
def test_removes_non_matching_mode(self):
|
|
141
|
+
content = "{{IF_MODE_FULL}}\nfull content\n{{END_IF_MODE_FULL}}\nrest"
|
|
142
|
+
result = process_mode_blocks(content, "lite", False)
|
|
143
|
+
assert "full content" not in result
|
|
144
|
+
assert "rest" in result
|
|
145
|
+
|
|
146
|
+
def test_init_done_keeps_init_done_block(self):
|
|
147
|
+
content = "{{IF_INIT_DONE}}\ninit done\n{{END_IF_INIT_DONE}}\n"
|
|
148
|
+
result = process_mode_blocks(content, "standard", True)
|
|
149
|
+
assert "init done" in result
|
|
150
|
+
|
|
151
|
+
def test_init_done_removes_init_needed_block(self):
|
|
152
|
+
content = "{{IF_INIT_NEEDED}}\nneed init\n{{END_IF_INIT_NEEDED}}\n"
|
|
153
|
+
result = process_mode_blocks(content, "standard", True)
|
|
154
|
+
assert "need init" not in result
|
|
155
|
+
|
|
156
|
+
def test_init_not_done_keeps_init_needed_block(self):
|
|
157
|
+
content = "{{IF_INIT_NEEDED}}\nneed init\n{{END_IF_INIT_NEEDED}}\n"
|
|
158
|
+
result = process_mode_blocks(content, "standard", False)
|
|
159
|
+
assert "need init" in result
|
|
160
|
+
|
|
161
|
+
def test_multiple_modes(self):
|
|
162
|
+
content = (
|
|
163
|
+
"{{IF_MODE_LITE}}\nlite\n{{END_IF_MODE_LITE}}\n"
|
|
164
|
+
"{{IF_MODE_STANDARD}}\nstandard\n{{END_IF_MODE_STANDARD}}\n"
|
|
165
|
+
"{{IF_MODE_FULL}}\nfull\n{{END_IF_MODE_FULL}}\n"
|
|
166
|
+
)
|
|
167
|
+
result = process_mode_blocks(content, "standard", False)
|
|
168
|
+
assert "lite" not in result
|
|
169
|
+
assert "standard" in result
|
|
170
|
+
assert "full" not in result
|
|
171
|
+
|
|
172
|
+
|
|
173
|
+
class TestDeterminePipelineMode:
|
|
174
|
+
def test_low_complexity(self):
|
|
175
|
+
assert determine_pipeline_mode("low") == "lite"
|
|
176
|
+
|
|
177
|
+
def test_medium_complexity(self):
|
|
178
|
+
assert determine_pipeline_mode("medium") == "standard"
|
|
179
|
+
|
|
180
|
+
def test_high_complexity(self):
|
|
181
|
+
assert determine_pipeline_mode("high") == "full"
|
|
182
|
+
|
|
183
|
+
def test_critical_complexity(self):
|
|
184
|
+
assert determine_pipeline_mode("critical") == "full"
|
|
185
|
+
|
|
186
|
+
def test_unknown_complexity(self):
|
|
187
|
+
assert determine_pipeline_mode("unknown") == "standard"
|
|
188
|
+
|
|
189
|
+
def test_none_complexity(self):
|
|
190
|
+
assert determine_pipeline_mode(None) == "standard"
|