Keep notification prefix private #40

Merged

barrettruth merged 1 commit from chore/private-notification-prefix into main

2026-05-05 16:41:25 +00:00

barrettruth commented

2026-05-05 16:35:06 +00:00

Owner

Problem

The notification-prefix cleanup centralized the prefix by exposing M.prefix from import-cost.util. That makes an internal formatting detail look like public module surface.

Solution

Keep the prefix as a private local in util.lua and route the GitHub-source migration warning through util.log(...).

Verification

nix develop .#ci --command stylua --check lua/import-cost/util.lua lua/import-cost/migration.lua
nix develop .#ci --command selene --display-style quiet lua/import-cost/util.lua lua/import-cost/migration.lua
nix develop .#ci --command just lint
git diff --check

## Problem The notification-prefix cleanup centralized the prefix by exposing `M.prefix` from `import-cost.util`. That makes an internal formatting detail look like public module surface. ## Solution Keep the prefix as a private local in `util.lua` and route the GitHub-source migration warning through `util.log(...)`. ## Verification - `nix develop .#ci --command stylua --check lua/import-cost/util.lua lua/import-cost/migration.lua` - `nix develop .#ci --command selene --display-style quiet lua/import-cost/util.lua lua/import-cost/migration.lua` - `nix develop .#ci --command just lint` - `git diff --check`

barrettruth added 1 commit

2026-05-05 16:35:06 +00:00

Keep notification prefix private

quality / Lint (pull_request) Successful in 5s

Details

quality / Format (pull_request) Successful in 9s

Details

70d08d4f3b