Adds dimensions for session-discovered patterns the audit was missing:

- legacy-log-package — catches `"dappco.re/go/log"` import regression.
  Today's go-ml `coreerr.E → core.E` sweep removed 237 instances; nothing
  prevented a regression until this dim. Smoke: go-ml=30, go-ai=2.
- service-usage-example — files declaring `NewService(` must include
  the literal `// Usage example:` marker. Mantis #1383 convention test
  was per-package; this makes the rule ecosystem-wide. Smoke: go-ml=1.
- service-canonical-shape — packages with `NewService(` must also
  declare `Register(c *core.Core) core.Result` or `RegisterCore` (name
  divergence allowed where Register collides with existing
  registration funcs, e.g. inference.Register(Backend)). Mantis #1336.
  Smoke: go-ml=1.
- error-wrap-antipattern — catches `core.NewError(x.Error())` two-stage
  wrapping that loses the original Result chain. Spotted today in
  BookState demo code. Smoke: go-ai=20, go-ml=4.

Together: 58 additional findings across go-ai + go-ml that the prior
audit hid. go-inference + go-mlx + core/go clean on all four. Initial
regex caught NewServiceRuntime false positive in core/go; tightened to
`^func NewService(` (open paren) to exclude generic ServiceRuntime
helpers.

Co-Authored-By: Virgil <virgil@lethean.io>