M.V. Hutz
a3ee34732e
## Description The old architecture used a monolithic `main()` with a custom arg parser, an event-emitter-based runtime, and a plugin system for optional features. This PR rewrites the CLI and internal architecture to be modular, extensible, and built around a registry of interchangeable components. - Replace custom CLI arg parsing with Cobra subcommands (`convert`, `reduce`, `engine list`). - Introduce a registry system (`internal/registry`) for marshalers, codecs, and engines, with BFS-based conversion path resolution. - Add type-erased adapter layer (`internal/cli`) with `Repr`, `Engine`, `Process`, `Marshaler`, and `Conversion` interfaces wrapping generic `pkg/` types. - Replace the event-emitter-based `Runtime` with a simpler `Engine`/`Process` model (`pkg/engine`). - Add generic `Codec[T, U]` and `Marshaler[T]` interfaces (`pkg/codec`). - Merge `saccharine/token` sub-package into `saccharine` and rename scanner functions from `parse*` to `scan*`. - Make saccharine-to-lambda conversion bidirectional (encode and decode). - Add `lambda.Marshaler` and `saccharine.Marshaler` implementing `codec.Marshaler`. - Remove old infrastructure: `pkg/runtime`, `pkg/expr`, `internal/plugins`, `internal/statistics`. - Add `make lint` target and update golangci-lint config. ### Decisions - Cobra was chosen for the CLI framework to support nested subcommands and standard flag handling. - The registry uses BFS to find conversion paths between representations, allowing multi-hop conversions without hardcoding routes. - Type erasure via `cli.Repr` (wrapping `any`) enables the registry to work with heterogeneous types while keeping `pkg/` generics type-safe. - The old plugin/event system was removed entirely rather than adapted, since the new `Process` model can support hooks differently in the future. ## Benefits - Subcommands make the CLI self-documenting and easier to extend with new functionality. - The registry pattern decouples representations, conversions, and engines, making it trivial to add new ones. - BFS conversion routing means adding a single codec automatically enables transitive conversions. - Simpler `Engine`/`Process` model reduces complexity compared to the event-emitter runtime. - Consolidating the `token` sub-package reduces import depth and package sprawl. ## Checklist - [x] Code follows conventional commit format. - [x] Branch follows naming convention (`<type>/<description>`). Always use underscores. - [ ] Tests pass (if applicable). - [ ] Documentation updated (if applicable). Reviewed-on: #41 Co-authored-by: M.V. Hutz <git@maximhutz.me> Co-committed-by: M.V. Hutz <git@maximhutz.me>
56 lines
1.6 KiB
Makefile
56 lines
1.6 KiB
Makefile
BINARY_NAME=lambda
|
|
TEST=simple
|
|
|
|
.PHONY: help build run profile explain graph docs test bench lint clean
|
|
.DEFAULT_GOAL := help
|
|
.SILENT:
|
|
|
|
help:
|
|
echo "Available targets:"
|
|
echo " build - Build the lambda executable"
|
|
echo " run - Build and run the lambda runtime (use TEST=<name> to specify sample)"
|
|
echo " profile - Build and run with CPU profiling enabled"
|
|
echo " explain - Build and run with explanation mode and profiling"
|
|
echo " graph - Generate and open CPU profile visualization"
|
|
echo " docs - Start local godoc server on port 6060"
|
|
echo " test - Run tests for all samples"
|
|
echo " bench - Run benchmarks for all samples"
|
|
echo " lint - Run golangci-lint on all packages"
|
|
echo " clean - Remove all build artifacts"
|
|
|
|
build:
|
|
go build -o ${BINARY_NAME} ./cmd/lambda
|
|
chmod +x ${BINARY_NAME}
|
|
|
|
run: build
|
|
./${BINARY_NAME} -s -f ./tests/$(TEST).test -o program.out
|
|
|
|
profile: build
|
|
./${BINARY_NAME} -p profile/cpu.prof -f ./tests/$(TEST).test -o program.out
|
|
|
|
explain: build
|
|
./${BINARY_NAME} -x -p profile/cpu.prof -f ./tests/$(TEST).test -o program.out > explain.out
|
|
|
|
graph:
|
|
go tool pprof -raw -output=profile/cpu.raw profile/cpu.prof
|
|
go tool pprof -svg profile/cpu.prof > profile/cpu.svg
|
|
echo ">>> View at 'file://$(PWD)/profile/cpu.svg'"
|
|
|
|
docs:
|
|
echo ">>> View at 'http://localhost:6060/pkg/git.maximhutz.com/max/lambda/'"
|
|
go run golang.org/x/tools/cmd/godoc@latest -http=:6060
|
|
|
|
test:
|
|
go test -v ./cmd/lambda
|
|
|
|
bench:
|
|
go test -bench=. -benchtime=10x -cpu=4 ./cmd/lambda
|
|
|
|
lint:
|
|
go run github.com/golangci/golangci-lint/v2/cmd/golangci-lint@latest run ./...
|
|
|
|
clean:
|
|
rm -f ${BINARY_NAME}
|
|
rm -f program.out
|
|
rm -rf profile/
|