refactor: add more detailed request errors to Project

This commit is contained in:
CJ van den Berg 2026-02-19 09:47:11 +01:00
parent 7dc3e531c6
commit 5bc7f4c2e8
Signed by: neurocyte
GPG key ID: 8EB1E1BB660E3FB9

View file

@ -55,7 +55,17 @@ const Self = @This();
const OutOfMemoryError = error{OutOfMemory}; const OutOfMemoryError = error{OutOfMemory};
const SpawnError = (OutOfMemoryError || error{ThespianSpawnFailed}); const SpawnError = (OutOfMemoryError || error{ThespianSpawnFailed});
pub const RequestError = error{InvalidRequest} || OutOfMemoryError || cbor.Error; pub const RequestError = error{
InvalidMostRecentFileRequest,
InvalidNewOrModifiedFilesRequest,
InvalidQueryNewOrModifiedFilesRequest,
InvalidRequestNewOrModifiedFilesRequest,
InvalidRecentFilesRequest,
InvalidQueryRecentFilesRequest,
InvalidGetMruPositionRequest,
InvalidVcsStatusRequest,
InvalidTasksRequest,
} || OutOfMemoryError || cbor.Error;
pub const StartLspError = (error{ ThespianSpawnFailed, Timeout, InvalidLspCommand } || LspError || OutOfMemoryError || cbor.Error); pub const StartLspError = (error{ ThespianSpawnFailed, Timeout, InvalidLspCommand } || LspError || OutOfMemoryError || cbor.Error);
pub const LspError = (error{ NoLsp, LspFailed } || OutOfMemoryError || std.Io.Writer.Error); pub const LspError = (error{ NoLsp, LspFailed } || OutOfMemoryError || std.Io.Writer.Error);
pub const GitError = error{InvalidGitResponse}; pub const GitError = error{InvalidGitResponse};
@ -386,7 +396,7 @@ inline fn sort_by_mtime(T: type, items: []T) void {
} }
pub fn request_n_most_recent_file(self: *Self, from: tp.pid_ref, n: usize) RequestError!void { pub fn request_n_most_recent_file(self: *Self, from: tp.pid_ref, n: usize) RequestError!void {
if (n >= self.files.items.len) return error.InvalidRequest; if (n >= self.files.items.len) return error.InvalidMostRecentFileRequest;
const file_path = if (self.files.items.len > 0) self.files.items[n].path else null; const file_path = if (self.files.items.len > 0) self.files.items[n].path else null;
from.send(.{file_path}) catch |e| from.send(.{file_path}) catch |e|
std.log.err("send request_n_most_recent_file failed: {t}", .{e}); std.log.err("send request_n_most_recent_file failed: {t}", .{e});
@ -415,7 +425,7 @@ fn simple_query_new_or_modified_files(self: *Self, from: tp.pid_ref, max: usize,
while (n < query.len) : (n += 1) matches[n] = idx + n; while (n < query.len) : (n += 1) matches[n] = idx + n;
from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, file.path, file.type, file.icon, file.color, file.vcs_status, matches }) catch |e| { from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, file.path, file.type, file.icon, file.color, file.vcs_status, matches }) catch |e| {
std.log.err("send new_or_modified_files failed: {t}", .{e}); std.log.err("send new_or_modified_files failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidNewOrModifiedFilesRequest;
}; };
i += 1; i += 1;
if (i >= max) return i; if (i >= max) return i;
@ -485,7 +495,7 @@ pub fn query_new_or_modified_files(self: *Self, from: tp.pid_ref, max: usize, qu
for (matches.items[0..@min(max, matches.items.len)]) |match| for (matches.items[0..@min(max, matches.items.len)]) |match|
from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, match.path, match.type, match.icon, match.color, match.vcs_status, match.matches }) catch |e| { from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, match.path, match.type, match.icon, match.color, match.vcs_status, match.matches }) catch |e| {
std.log.err("send new_or_modified_files failed: {t}", .{e}); std.log.err("send new_or_modified_files failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidQueryNewOrModifiedFilesRequest;
}; };
return @min(max, matches.items.len); return @min(max, matches.items.len);
} }
@ -495,7 +505,7 @@ pub fn request_new_or_modified_files(self: *Self, from: tp.pid_ref, max: usize)
for (self.new_or_modified_files.items, 0..) |file, i| { for (self.new_or_modified_files.items, 0..) |file, i| {
from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, file.path, file.type, file.icon, file.color, file.vcs_status }) catch |e| { from.send(.{ "PRJ", "new_or_modified_files", self.longest_new_or_modified_file_path, file.path, file.type, file.icon, file.color, file.vcs_status }) catch |e| {
std.log.err("send navigate failed: {t}", .{e}); std.log.err("send navigate failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidRequestNewOrModifiedFilesRequest;
}; };
if (i >= max) return; if (i >= max) return;
} }
@ -513,7 +523,7 @@ fn simple_query_recent_files(self: *Self, from: tp.pid_ref, max: usize, query: [
while (n < query.len) : (n += 1) matches[n] = idx + n; while (n < query.len) : (n += 1) matches[n] = idx + n;
from.send(.{ "PRJ", "recent", self.longest_file_path, file.path, file.type, file.icon, file.color, matches }) catch |e| { from.send(.{ "PRJ", "recent", self.longest_file_path, file.path, file.type, file.icon, file.color, matches }) catch |e| {
std.log.err("send navigate failed: {t}", .{e}); std.log.err("send navigate failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidRecentFilesRequest;
}; };
i += 1; i += 1;
if (i >= max) return i; if (i >= max) return i;
@ -572,7 +582,7 @@ pub fn query_recent_files(self: *Self, from: tp.pid_ref, max: usize, query_: []c
for (matches.items[0..@min(max, matches.items.len)]) |match| for (matches.items[0..@min(max, matches.items.len)]) |match|
from.send(.{ "PRJ", "recent", self.longest_file_path, match.path, match.type, match.icon, match.color, match.matches }) catch |e| { from.send(.{ "PRJ", "recent", self.longest_file_path, match.path, match.type, match.icon, match.color, match.matches }) catch |e| {
std.log.err("send navigate failed: {t}", .{e}); std.log.err("send navigate failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidQueryRecentFilesRequest;
}; };
return @min(max, matches.items.len); return @min(max, matches.items.len);
} }
@ -808,10 +818,10 @@ fn update_mru_internal(self: *Self, file_path: []const u8, mtime: i128, row: usi
pub fn get_mru_position(self: *Self, from: tp.pid_ref, file_path: []const u8) RequestError!void { pub fn get_mru_position(self: *Self, from: tp.pid_ref, file_path: []const u8) RequestError!void {
for (self.files.items) |*file| { for (self.files.items) |*file| {
if (!std.mem.eql(u8, file.path, file_path)) continue; if (!std.mem.eql(u8, file.path, file_path)) continue;
from.send(.{ file.pos.row + 1, file.pos.col + 1 }) catch return error.InvalidRequest; from.send(.{ file.pos.row + 1, file.pos.col + 1 }) catch return error.InvalidGetMruPositionRequest;
return; return;
} }
from.send(.{"none"}) catch return error.InvalidRequest; from.send(.{"none"}) catch return error.InvalidGetMruPositionRequest;
} }
pub fn request_vcs_status(self: *Self, from: tp.pid_ref) RequestError!void { pub fn request_vcs_status(self: *Self, from: tp.pid_ref) RequestError!void {
@ -822,7 +832,7 @@ pub fn request_vcs_status(self: *Self, from: tp.pid_ref) RequestError!void {
if (self.status_request) |_| return; if (self.status_request) |_| return;
self.status_request = from.clone(); self.status_request = from.clone();
}, },
else => return error.InvalidRequest, else => return error.InvalidVcsStatusRequest,
}, },
.running => { .running => {
if (self.status_request) |_| return; if (self.status_request) |_| return;
@ -859,7 +869,7 @@ pub fn request_tasks(self: *Self, from: tp.pid_ref) RequestError!void {
try cbor.writeValue(writer, task.command); try cbor.writeValue(writer, task.command);
from.send_raw(.{ .buf = message.written() }) catch |e| { from.send_raw(.{ .buf = message.written() }) catch |e| {
std.log.err("send navigate failed: {t}", .{e}); std.log.err("send navigate failed: {t}", .{e});
return error.InvalidRequest; return error.InvalidTasksRequest;
}; };
} }