docs: document remaining packages and simplify AST types #45

mvhutz · 2026-02-10T01:15:06Z

mvhutz commented

2026-02-10 01:15:06 +00:00

Summary

Added doc comments across the codebase: pkg/lambda, pkg/saccharine, pkg/codec, pkg/engine, pkg/iterator, pkg/set, pkg/convert, internal/registry, and cmd/lambda.
Made lambda and saccharine expression structs use public fields instead of getters, matching go/ast conventions.
Removed superfluous constructors for saccharine and lambda expression/statement types in favor of struct literals.
Consolidated saccharine token constructors into a single NewToken function.
Removed the unused trace package.

Test plan

go build ./... passes.
go test ./... passes.
Verify go doc output renders correctly for documented packages.

## Summary - Added doc comments across the codebase: `pkg/lambda`, `pkg/saccharine`, `pkg/codec`, `pkg/engine`, `pkg/iterator`, `pkg/set`, `pkg/convert`, `internal/registry`, and `cmd/lambda`. - Made lambda and saccharine expression structs use public fields instead of getters, matching `go/ast` conventions. - Removed superfluous constructors for saccharine and lambda expression/statement types in favor of struct literals. - Consolidated saccharine token constructors into a single `NewToken` function. - Removed the unused `trace` package. ## Test plan - [x] `go build ./...` passes. - [x] `go test ./...` passes. - [ ] Verify `go doc` output renders correctly for documented packages.

mvhutz added 15 commits 2026-02-10 01:15:06 +00:00

docs: codec b9c0d5a6c7

docs: registry b259ab0125

feat: removed trace package 375148eae5

docs: iterator 1fcffb2ea4

docs: cmd/lambda 4171f0ccb2

docs: set 2a028d95ec

docs: registry 98b327103f

docs: process 4201225ce9

docs: token d24cbd9d86

feat: no constructors for expressions and statements for saccharine c4ba80924a

docs: saccharine expressions/stmts, private tokenIterator c6186d9c80