Skip to content

Commit 67e6f62

Browse files
committed
docs(ecc): synthesize PR #36 parser-provider paths onto merged
Path-scoped replay from ecc-tools/periscope-1785391024054 (PR #36): adopt provider_migration.go and concrete testdata paths. Reject wholesale merge — PR #36 branch would revert periscope rename and amputate PR #30 command scaffolds.
1 parent 11a985b commit 67e6f62

1 file changed

Lines changed: 14 additions & 12 deletions

File tree

.claude/commands/add-new-parser-or-provider.md

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,17 @@ Use this workflow when working on **add-new-parser-or-provider** in `periscope`.
1010

1111
## Goal
1212

13-
Adds support for a new agent/provider/parser, including implementation, integration, and test coverage.
13+
Adds support for a new agent, data source, or provider to the system, including schema detection, discovery, and integration with sync and test coverage.
1414

1515
## Common Files
1616

17-
- `internal/parser/*_provider.go`
18-
- `internal/parser/*.go`
19-
- `internal/parser/testdata/*/*.json`
20-
- `internal/parser/*_test.go`
17+
- `internal/parser/<agent>_provider.go`
18+
- `internal/parser/<agent>.go`
19+
- `internal/parser/types.go`
2120
- `internal/parser/provider.go`
22-
- `internal/sync/engine.go`
21+
- `internal/parser/provider_migration.go`
22+
- `internal/parser/testdata/<agent>/*`
23+
- `internal/parser/<agent>_test.go`
2324

2425
## Suggested Sequence
2526

@@ -30,13 +31,14 @@ Adds support for a new agent/provider/parser, including implementation, integrat
3031

3132
## Typical Commit Signals
3233

33-
- Add new parser/provider Go files (e.g., internal/parser/<provider>_provider.go, internal/parser/<provider>.go).
34-
- Update provider registry or discovery (e.g., internal/parser/provider.go).
35-
- Add or update test data and test cases (e.g., internal/parser/testdata/<provider>/*.json, internal/parser/<provider>_test.go).
36-
- Integrate with sync engine if needed (e.g., internal/sync/engine.go).
37-
- Update documentation and format sources (e.g., docs/internal/session-format-sources.md, docs/configuration.md).
34+
- Implement parser/provider logic in `internal/parser` (e.g., `<agent>_provider.go`, `<agent>.go`).
35+
- Add or update types in `internal/parser/types.go`.
36+
- Update provider registration in `internal/parser/provider.go` and/or `provider_migration.go`.
37+
- Add test fixtures and coverage in `internal/parser/testdata/<agent>/*` and `<agent>_test.go`.
38+
- Integrate with sync engine (`internal/sync/engine.go`, `engine_test.go`, `integration_test.go`).
39+
- Update documentation and format sources (e.g., `docs/internal/session-format-sources.md`, `docs/configuration.md`).
3840

3941
## Notes
4042

4143
- Treat this as a scaffold, not a hard-coded script.
42-
- Update the command if the workflow evolves materially.
44+
- Update the command if the workflow evolves materially.

0 commit comments

Comments
 (0)