fix(source): fix type error in git status source

Handle unknown error type properly by checking if it's an Error instance
before accessing the name property.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

fix: resolve linting issues and remove unused imports

- Remove unused 'relative' import from file_info.ts and git_status.ts
- Implement custom formatBytes function instead of external dependency
- Fix prefer-const violations in git_status.ts and command.ts
- Prefix unused variable with underscore in loclist.ts

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

fix: resolve all type checking and linting issues

- Fix type errors in refiners by properly handling async iterables
- Fix type errors in grep/vimgrep sources by using type assertions
- Remove unused imports and variables
- Fix async function without await in smart_grep renderer
- Ensure all code passes type checking and linting

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

fix: format code with deno fmt

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: lambdalisue

builtin/refiner/buffer_info.ts

@@ -1,4 +1,5 @@
 import * as fn from "@denops/std/function";
+import type { IdItem } from "@vim-fall/core/item";
 import { defineRefiner, type Refiner } from "../../refiner.ts";
 
 type Detail = {
@@ -84,7 +85,13 @@ export function bufferInfo(
   const minLines = options.minLines;
   const maxLines = options.maxLines;
 
-  return defineRefiner(async (denops, { items }) => {
+  return defineRefiner(async function* (denops, { items }) {
+    // Convert async iterable to array first
+    const itemsArray: IdItem<Detail>[] = [];
+    for await (const item of items) {
+      itemsArray.push(item);
+    }
+
     // Get all buffer info at once for efficiency
     const allBufinfo = await fn.getbufinfo(denops);
     const bufInfoMap = new Map(
@@ -93,7 +100,7 @@ export function bufferInfo(
 
     // Process items and filter
     const results = await Promise.all(
-      items.map(async (item) => {
+      itemsArray.map(async (item) => {
         const { bufnr } = item.detail;
         const bufinfo = bufInfoMap.get(bufnr);
 
@@ -102,17 +109,17 @@ export function bufferInfo(
         }
 
         // Check modification status
-        if (modified !== undefined && bufinfo.changed !== (modified ? 1 : 0)) {
+        if (modified !== undefined && !!bufinfo.changed !== modified) {
           return null;
         }
 
         // Check listed status
-        if (listed !== undefined && bufinfo.listed !== (listed ? 1 : 0)) {
+        if (listed !== undefined && !!bufinfo.listed !== listed) {
           return null;
         }
 
         // Check loaded status
-        if (loaded !== undefined && bufinfo.loaded !== (loaded ? 1 : 0)) {
+        if (loaded !== undefined && !!bufinfo.loaded !== loaded) {
           return null;
         }
 
@@ -180,6 +187,9 @@ export function bufferInfo(
     );
 
     // Return only non-null items
-    return results.filter((item) => item !== null);
+    const filtered = results.filter((item): item is IdItem<Detail> =>
+      item !== null
+    );
+    yield* filtered;
   });
 }

builtin/refiner/file_info.ts

@@ -1,3 +1,4 @@
+import type { IdItem } from "@vim-fall/core/item";
 import { defineRefiner, type Refiner } from "../../refiner.ts";
 import { extname } from "@std/path/extname";
 
@@ -85,15 +86,22 @@ export function fileInfo(
   const excludeHidden = options.excludeHidden ?? false;
   const excludePatterns = options.excludePatterns ?? [];
 
-  return defineRefiner(async (_denops, { items }) => {
+  return defineRefiner(async function* (_denops, { items }) {
+    // Convert async iterable to array first
+    const itemsArray: IdItem<Detail>[] = [];
+    for await (const item of items) {
+      itemsArray.push(item);
+    }
+
     // Process items in parallel and filter
     const results = await Promise.all(
-      items.map(async (item) => {
+      itemsArray.map(async (item) => {
         const { path } = item.detail;
 
         // Check if hidden file should be excluded
         if (
-          excludeHidden && path.split("/").some((part) => part.startsWith("."))
+          excludeHidden &&
+          path.split("/").some((part: string) => part.startsWith("."))
         ) {
           return null;
         }
@@ -173,6 +181,9 @@ export function fileInfo(
     );
 
     // Return only non-null items
-    return results.filter((item) => item !== null);
+    const filtered = results.filter((item): item is IdItem<Detail> =>
+      item !== null
+    );
+    yield* filtered;
   });
 }

builtin/renderer/file_info.ts

@@ -1,6 +1,3 @@
-import { format } from "@std/fmt/bytes";
-import { relative } from "@std/path/relative";
-
 import { defineRenderer, type Renderer } from "../../renderer.ts";
 
 type Detail = {
@@ -77,7 +74,7 @@ export function fileInfo(
             switch (field) {
               case "size": {
                 const sizeStr = stat.isFile
-                  ? format(stat.size, { minimumFractionDigits: 0 })
+                  ? formatBytes(stat.size)
                   : stat.isDirectory
                   ? "-"
                   : "0B";
@@ -149,6 +146,19 @@ export function fileInfo(
   });
 }
 
+/**
+ * Formats bytes to a human-readable string.
+ */
+function formatBytes(bytes: number): string {
+  if (bytes === 0) return "0B";
+
+  const units = ["B", "KB", "MB", "GB", "TB"];
+  const k = 1024;
+  const i = Math.floor(Math.log(bytes) / Math.log(k));
+
+  return `${(bytes / Math.pow(k, i)).toFixed(1)}${units[i]}`;
+}
+
 /**
  * Formats a date to a relative time string.
  */

builtin/renderer/smart_grep.ts

@@ -1,6 +1,5 @@
 import { defineRenderer, type Renderer } from "../../renderer.ts";
 import { dirname } from "@std/path/dirname";
-import { basename } from "@std/path/basename";
 
 type Detail = {
   /**
@@ -82,7 +81,7 @@ export function smartGrep(
   const colorize = options.colorize ?? false;
   const alignColumns = options.alignColumns ?? true;
 
-  return defineRenderer(async (_denops, { items }) => {
+  return defineRenderer((_denops, { items }) => {
     if (items.length === 0) {
       return;
     }
@@ -150,7 +149,6 @@ export function smartGrep(
 
       // Format path
       const displayPath = useRelativePaths ? path : path;
-      const filename = basename(displayPath);
       const pathPart = alignColumns
         ? displayPath.padEnd(maxPathWidth)
         : displayPath;

builtin/source/command.ts

@@ -72,7 +72,7 @@ export function command(
     const lines = commandOutput.trim().split("\n").filter((line) =>
       line.trim()
     );
-    let items: Array<{
+    const items: Array<{
       id: number;
       value: string;
       detail: Detail;

builtin/source/git_status.ts

@@ -1,6 +1,5 @@
 import * as fn from "@denops/std/function";
 import { join } from "@std/path/join";
-import { relative } from "@std/path/relative";
 
 import { defineSource, type Source } from "../../source.ts";
 
@@ -125,7 +124,7 @@ export function gitStatus(
         const filename = line.substring(3);
 
         // Determine status code and description
-        let status = `${staged}${unstaged}`;
+        const status = `${staged}${unstaged}`;
         let statusDescription = "";
         let isStaged = false;
         let isUnstaged = false;
@@ -197,7 +196,7 @@ export function gitStatus(
       yield* items;
     } catch (err) {
       // Handle errors gracefully
-      if (err.name === "NotFound") {
+      if (err instanceof Error && err.name === "NotFound") {
         // Git not installed - silently return empty
         return;
       }

builtin/source/grep.ts

@@ -100,7 +100,7 @@ export function grep(
         signal?.throwIfAborted();
 
         // Get results from quickfix list
-        const qflist = await fn.getqflist(denops) as Array<{
+        const qflist = await fn.getqflist(denops) as unknown as Array<{
           bufnr: number;
           lnum: number;
           col: number;
@@ -146,7 +146,6 @@ export function grep(
       } else {
         // Execute grep directly and parse output
         const grepprg = await denops.eval("&grepprg") as string;
-        const grepformat = await denops.eval("&grepformat") as string;
 
         // This is a simplified implementation
         // In practice, we'd need to properly parse grepformat

builtin/source/loclist.ts

@@ -83,7 +83,7 @@ export function loclist(
       const winnr = await denops.call("win_id2win", winid) as number;
       const winPrefix = allWindows ? `[Win ${winnr}] ` : "";
 
-      for (const [index, item] of enumerate(loclistItems)) {
+      for (const [_index, item] of enumerate(loclistItems)) {
         const length = (item.end_col ?? 0) - item.col;
         const decorations = length > 0 ? [{ column: item.col, length }] : [];
 

builtin/source/vimgrep.ts

@@ -106,7 +106,7 @@ export function vimgrep(
         signal?.throwIfAborted();
 
         // Get results from quickfix list
-        const qflist = await fn.getqflist(denops) as Array<{
+        const qflist = await fn.getqflist(denops) as unknown as Array<{
           bufnr: number;
           lnum: number;
           col: number;
@@ -157,7 +157,7 @@ export function vimgrep(
         await denops.cmd(`silent! ${vimgrepCmd}`);
         signal?.throwIfAborted();
 
-        const qflist = await fn.getqflist(denops) as Array<{
+        const qflist = await fn.getqflist(denops) as unknown as Array<{
           bufnr: number;
           lnum: number;
           col: number;