chore: refine code

.github/workflows/ci.yml

@@ -14,7 +14,6 @@ jobs:
           version: latest
           args: --check .
 
-
   docs:
     runs-on: ubuntu-latest
     name: pandoc to vimdoc
@@ -28,7 +27,7 @@ jobs:
           treesitter: true
       - uses: stefanzweifel/git-auto-commit-action@v4
         with:
-          commit_message: 'chore(doc): auto generate docs'
+          commit_message: "chore(doc): auto generate docs"
           commit_user_name: "github-actions[bot]"
           commit_user_email: "github-actions[bot]@users.noreply.github.com"
           commit_author: "github-actions[bot] <github-actions[bot]@users.noreply.github.com>"

lua/guard/events.lua

@@ -1,9 +1,10 @@
 local api, uv = vim.api, vim.uv
 local group = api.nvim_create_augroup('Guard', { clear = true })
+local au = api.nvim_create_autocmd
 local M = {}
 
 function M.attach_to_buf(buf)
-  api.nvim_create_autocmd('BufWritePre', {
+  au('BufWritePre', {
     group = group,
     buffer = buf,
     callback = function(opt)
@@ -16,7 +17,7 @@ function M.attach_to_buf(buf)
 end
 
 function M.watch_ft(ft)
-  api.nvim_create_autocmd('FileType', {
+  au('FileType', {
     group = group,
     pattern = ft,
     callback = function(args)
@@ -35,7 +36,7 @@ function M.watch_ft(ft)
 end
 
 function M.create_lspattach_autocmd(fmt_on_save)
-  api.nvim_create_autocmd('LspAttach', {
+  au('LspAttach', {
     group = group,
     callback = function(args)
       local client = vim.lsp.get_client_by_id(args.data.client_id)
@@ -65,7 +66,7 @@ end
 
 local debounce_timer = nil
 function M.register_lint(ft, events)
-  api.nvim_create_autocmd('FileType', {
+  au('FileType', {
     pattern = ft,
     group = group,
     callback = function(args)
@@ -87,7 +88,7 @@ function M.register_lint(ft, events)
       end
       for _, ev in ipairs(events) do
         if ev == 'User GuardFmt' then
-          api.nvim_create_autocmd('User', {
+          au('User', {
             group = group,
             pattern = 'GuardFmt',
             callback = function(opt)
@@ -97,7 +98,7 @@ function M.register_lint(ft, events)
             end,
           })
         else
-          api.nvim_create_autocmd(ev, {
+          au(ev, {
             group = group,
             buffer = args.buf,
             callback = cb,

lua/guard/format.lua

@@ -14,6 +14,7 @@ local api = vim.api
 local spawn = require('guard.spawn')
 local util = require('guard.util')
 local filetype = require('guard.filetype')
+local iter, filter = vim.iter, vim.tbl_filter
 
 local function save_views(bufnr)
   local views = {}
@@ -87,29 +88,29 @@ local function do_fmt(buf)
   local fname, startpath, root_dir, cwd = util.buf_get_info(buf)
 
   -- handle execution condition
-  fmt_configs = vim.tbl_filter(function(config)
+  fmt_configs = filter(function(config)
     return util.should_run(config, buf, startpath, root_dir)
   end, fmt_configs)
 
   -- check if all cmds executable
-  local non_excutable = vim.tbl_filter(function(config)
+  local non_excutable = filter(function(config)
     return config.cmd and vim.fn.executable(config.cmd) ~= 1
   end, fmt_configs)
 
   if #non_excutable > 0 then
-    error(table.concat(
+    error(('%s not executable'):format(table.concat(
       vim.tbl_map(function(config)
         return config.cmd
       end, non_excutable),
       ', '
-    ) .. ' not executable')
+    )))
   end
 
   -- filter out "pure" and "impure" formatters
-  local pure = vim.iter(vim.tbl_filter(function(config)
+  local pure = iter(filter(function(config)
     return config.fn or (config.cmd and config.stdin)
   end, fmt_configs))
-  local impure = vim.iter(vim.tbl_filter(function(config)
+  local impure = iter(filter(function(config)
     return config.cmd and not config.stdin
   end, fmt_configs))
 

test/spawn_spec.lua

@@ -0,0 +1,14 @@
+---@diagnostic disable: undefined-field, undefined-global
+local spawn = require('guard.spawn')
+local same = assert.are.same
+
+describe('spawn module', function()
+  it('can spawn executables with stdin access', function()
+    coroutine.resume(coroutine.create(function()
+      local result = spawn.transform({ 'tac', '-s', '  ' }, {
+        stdin = true,
+      }, 'test1  test2  test3  ')
+      same(result, 'test3  test2  test1  ')
+    end))
+  end)
+end)