Getting error on import API module: field < copy >

[kevinm6]

Description

(First, great work on this plugin, I really like it and use everyday).

I tried to improve my nvim-tree config replacing some nvim-tree.lib with the relatives from the new Api module.
But on trying I’m always getting an error in import that module, specifically this field copy in api!

The error is : attempt to index field copy (a function value)

Neovim version

NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by brew@Monterey

Operating system and version

macOS 12.0.1

nvim-tree version

86b9da5

Minimal config

local ok, nvim_tree = pcall(require, "nvim-tree")
if not ok then return end

local api = require "nvim-tree.api"

local function trash_file()
  local node = api.get_node_under_cursor()

  -- code
  api.tree.reload()
end

local function vsplit_preview()
  local node = api.get_node_under_cursor()

  if node.link_to and not node.nodes then
    api.tree.open(node.link_to)
  else
    api.tree.open(node.absolute_path)
  end
  api.tree.focus()
end

Steps to reproduce

nvim

Expected behavior

No response

Actual behavior

No response

[kevinm6]

I’ve just try to fix-it myself checking the api module and the functions calls and I think I get the problem:

local Api = {
  tree = {},
  node = { navigate = { sibling = {}, git = {}, diagnostics = {} }, run = {}, open = {} },
  events = {},
  marks = { bulk = {}, navigate = {} },
  fs = { copy = {} },
  git = {},
  live_filter = {},
}

here copy is a table and is being populate by some of Api.fs calls like:

Api.fs.copy = inject_node(require("nvim-tree.actions.fs.copy-paste").copy)
Api.fs.copy.absolute_path = inject_node(require("nvim-tree.actions.fs.copy-paste").copy_absolute_path)
Api.fs.copy.filename = inject_node(require("nvim-tree.actions.fs.copy-paste").copy_filename)
Api.fs.copy.relative_path = inject_node(require("nvim-tree.actions.fs.copy-paste").copy_path)

but I think this
Api.fs.copy = inject_node(require("nvim-tree.actions.fs.copy-paste").copy)
should be
Api.fs.copy.copy = inject_node(require("nvim-tree.actions.fs.copy-paste").copy)
otherwise the call of it replace the copy = {} "main table” of the parent fs = { copy = {} }

[kyazdani42]

good catch :) thanks. You can use copy.name now.

[kyazdani42]

@kevinm6 use copy.node instead of copy.name. Copy.name is actually already copy.filename.