Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ These instructions have been tested on a clean machine to ensure a reliable setu

> **Note:** If the repository already includes `.env.example`, you only need to copy it to `.env`. Creating a new `.env.example` is only necessary if the file is missing.


5. **Start the development server:**
```bash
bun dev
Expand Down
62 changes: 28 additions & 34 deletions src/lib/api/github.ts
Original file line number Diff line number Diff line change
Expand Up @@ -137,21 +137,19 @@ export async function searchIssues(

const issues = data.items
.filter((item) => !item.pull_request)
.map(
(item): Issue => ({
number: item.number,
title: item.title,
body: item.body,
user: item.user,
labels: item.labels,
assignees: item.assignees,
comments_count: item.comments,
created_at: item.created_at,
updated_at: item.updated_at,
html_url: item.html_url,
state: item.state,
}),
);
.map((item): Issue => ({
number: item.number,
title: item.title,
body: item.body,
user: item.user,
labels: item.labels,
assignees: item.assignees,
comments_count: item.comments,
created_at: item.created_at,
updated_at: item.updated_at,
html_url: item.html_url,
state: item.state,
}));

allIssues.push(...issues);

Expand Down Expand Up @@ -190,15 +188,13 @@ export async function fetchIssueComments(

const data = await response.json();

return data.map(
(c: Record<string, unknown>): Comment => ({
id: c.id as number,
user: c.user as Comment['user'],
body: c.body as string,
created_at: c.created_at as string,
updated_at: c.updated_at as string,
}),
);
return data.map((c: Record<string, unknown>): Comment => ({
id: c.id as number,
user: c.user as Comment['user'],
body: c.body as string,
created_at: c.created_at as string,
updated_at: c.updated_at as string,
}));
}

export async function fetchIssueTimeline(
Expand All @@ -217,16 +213,14 @@ export async function fetchIssueTimeline(

const data: Record<string, unknown>[] = await response.json();

return data.map(
(e): TimelineEvent => ({
event: e.event as string,
created_at: e.created_at as string,
actor: e.actor as TimelineEvent['actor'],
source: e.source as TimelineEvent['source'],
commit_id: e.commit_id as string | undefined,
label: e.label as TimelineEvent['label'],
}),
);
return data.map((e): TimelineEvent => ({
event: e.event as string,
created_at: e.created_at as string,
actor: e.actor as TimelineEvent['actor'],
source: e.source as TimelineEvent['source'],
commit_id: e.commit_id as string | undefined,
label: e.label as TimelineEvent['label'],
}));
} catch {
return [];
}
Expand Down
11 changes: 11 additions & 0 deletions src/lib/utils/formatters.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,17 @@ describe('formatters utilities', () => {
expect(truncate('hello world', 6)).toBe('hello…');
});

it('never exceeds maxLen when the limit is 1', () => {
expect(truncate('hello', 1)).toBe('…');
});

it('handles zero or negative maxLen without producing longer output', () => {
// Previously truncate('hello', 0) returned 'hell…' — longer than the input
expect(truncate('hello', 0)).toBe('…');
expect(truncate('hello', -3)).toBe('…');
expect(truncate('', 0)).toBe('');
});

// ── complexityLabel ─────────────────────────────
it('returns correct complexity label', () => {
expect(complexityLabel(2)).toBe('Simple');
Expand Down
4 changes: 4 additions & 0 deletions src/lib/utils/formatters.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,10 @@ export function formatTimeAgo(ts: string | number): string {
* @returns The truncated string with an ellipsis, or the original string if it is short enough.
*/
export function truncate(str: string, maxLen: number): string {
if (str.length === 0) return str;
// The ellipsis alone is one character, so anything at or below 1 can only
// render the ellipsis itself
if (maxLen <= 1) return '…';
if (str.length <= maxLen) return str;
return str.slice(0, maxLen - 1) + '…';
}
Expand Down
Loading