Pre-requisites

• The issue is reproducible
• The aren't other issues similar to this problem.
• The aren't PRs tackling this problem.

NeoVim version

NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1693350652

Operative system

MacOS

OS version

Darwin air-m1 23.5.0 Darwin Kernel Version 23.5.0: Wed May 1 20:16:51 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T8103 arm64 arm Darwin

Behaviour

I was trying it out and noticed that its plugin fails to copy image to clipboard on macOS.

I debugged the problem and found that is failing due to argument to osascript been wrapped with an extra quote (') on this line:

Executing this command on Neovim yields the following result:

Removing the extra quotes around the last argument, running the command works and the image goes to the clipboard as expected:

I'm submitting this simple fix as a PR linked to this issue.
Thank you so much for creating this plugin.

Steps to reproduce

1. Open nvim with the minimal configuration, having copy = truein the freeze-code setup.
2. Type :Freeze
3. Upon completing the steps above, it will successfully produce a freeze.png, but shortly after, it will print a message “Error while copying screenshot to clipboard”

Minimal config

vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/]])

local lazypath = "/tmp/nvim/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
  {
    "AlejandroSuero/freeze-code.nvim",
    config = function()
      require("freeze-code").setup({
          copy = true
      })
    end,
    lazy = false,
    enabled = true,
  },
})

Code of conduct agreement

• I agree and I am not violating the CODE OF CONDUCT

Is your feature request related to a problem? Please describe

To have an installation method for freeze.

Describe the solution you'd like

• Check if freeze is installed vim.fn.exepath("freeze")
  • If not installed, check if golang is installed.
    • If not installed, grab the latest release from GitHub.
    • If installed, use go install github.com/charmbracelet/freeze@latest.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context or screenshots about the feature request here.

Is your feature request related to a problem? Please describe

Improve user experience by providing less time from command to notification.

Describe the solution you'd like

os.execute may cause issues if the command fails, vim.fn.system executes in
the background and it can be checked with vim.v.shell_error.

Also, change the execution of ./bin/<copy-command> to inline command declaration.
Like this:

local cmd_args = { ... }
vim.fn.system({ os_cmd, table.concat(cmd_args, " ") })