Barrett Ruth
29af2df858
Problem: `logger.log` positional args were hard to extend, and adding `sync` support for pre-block notifications required a clean API. Test stream completion had no user-visible signal. `setup_contest` could silently overwrite files when a user's `filename` config returned colliding paths. Solution: Replace `(msg, level, override)` with `(msg, LogOpts?)` where `LogOpts` carries `level`, `override`, and `sync`. Sync path calls `vim.notify` directly; async path uses `vim.schedule` as before. Add `on_done` callback to `scrape_all_tests`, fired via `on_exit` and surfaced as a "Loaded N tests." notification. Detect filename collisions in `proceed()` before touching the filesystem. Migrate all call sites.
28 lines
641 B
Lua
28 lines
641 B
Lua
local M = {}
|
|
|
|
---@class LogOpts
|
|
---@field level? integer
|
|
---@field override? boolean
|
|
---@field sync? boolean
|
|
|
|
---@param msg string
|
|
---@param opts? LogOpts
|
|
function M.log(msg, opts)
|
|
local debug = require('cp.config').get_config().debug or false
|
|
opts = opts or {}
|
|
local level = opts.level or vim.log.levels.INFO
|
|
local override = opts.override or false
|
|
local sync = opts.sync or false
|
|
if level >= vim.log.levels.WARN or override or debug then
|
|
local notify = function()
|
|
vim.notify(('[cp.nvim]: %s'):format(msg), level)
|
|
end
|
|
if sync then
|
|
notify()
|
|
else
|
|
vim.schedule(notify)
|
|
end
|
|
end
|
|
end
|
|
|
|
return M
|