fix: pass issueId to CommentClient.GetAllAsync in IssueDetailPage

[Copilot]

IssueDetailPage was fetching every comment in the database and filtering client-side — a query that degrades linearly with total comment volume.

Changes

• IssueDetailPage.razor — pass issueId: Id to CommentClient.GetAllAsync(); remove the client-side .Where() filter that was masking the missing parameter
• IssueDetailPageTests.cs — add assertion that GetAllAsync is called with the exact issue ID (not null), enforcing server-side filtering

// Before
var allComments = await CommentClient.GetAllAsync();
_comments = allComments.Where(c => c.Issue.Id.ToString() == Id);

// After
_comments = await CommentClient.GetAllAsync(issueId: Id);

CommentApiClient.GetAllAsync already appended ?issueId= to the request URL when the parameter was provided — it simply wasn't being passed.

[Copilot]

Pull request overview

This PR fixes IssueDetailPage comment loading to use server-side filtering by passing the current issue ID into CommentClient.GetAllAsync, avoiding a full comment-table scan and client-side filtering.

Changes:

• Update IssueDetailPage.razor to call CommentClient.GetAllAsync(issueId: Id) and remove the client-side .Where(...) filter.
• Add a bUnit test asserting the page calls GetAllAsync with the specific issue ID.
• Reformat src/Web/package-lock.json (appears unrelated to the stated PR purpose).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File	Description
tests/Web.Tests.Bunit/Components/Features/Issues/IssueDetailPageTests.cs	Adds a regression test ensuring GetAllAsync is invoked with the issue ID.
src/Web/Components/Features/Issues/IssueDetailPage.razor	Passes issueId into comment retrieval to enable server-side filtering.
src/Web/package-lock.json	Large formatting-only diff (indentation/whitespace changes).

[Copilot]

package-lock.json appears to have been entirely re-indented/reformatted without any accompanying dependency/version changes. This creates a very large, unrelated diff that can cause merge conflicts and makes the PR harder to review. Consider reverting this file (or regenerating it only if dependency changes are intended) and keeping lockfile updates in a separate, dedicated change.

[Copilot]

This test asserts that GetAllAsync is called with the issue ID, but it would still pass if the component also made an additional unfiltered call (e.g., GetAllAsync()/issueId: null), which would reintroduce the performance regression. Consider adding a negative assertion (e.g., DidNotReceive for issueId: null/empty) or asserting the total call count to fully enforce server-side filtering.

Suggested change

	.GetAllAsync(issue.Id.ToString(), Arg.Any<CancellationToken>());
	.GetAllAsync(issue.Id.ToString(), Arg.Any<CancellationToken>());
	_mockCommentClient.DidNotReceive()
	.GetAllAsync((string?)null, Arg.Any<CancellationToken>());
	_mockCommentClient.DidNotReceive()
	.GetAllAsync(string.Empty, Arg.Any<CancellationToken>());

[github-actions]

Test Results Summary

  6 files  ±0   33 suites  ±0   10s ⏱️ -1s
836 tests +1  836 ✅ +1  0 💤 ±0  0 ❌ ±0 
847 runs  +1  847 ✅ +1  0 💤 ±0  0 ❌ ±0 

Results for commit 66685eb. ± Comparison against base commit 7180786.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 55.53%. Comparing base (7180786) to head (66685eb).
⚠️ Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
...b/Components/Features/Issues/IssueDetailPage.razor	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
+ Coverage   55.52%   55.53%   +0.01%     
==========================================
  Files         124      124              
  Lines        2826     2825       -1     
  Branches      313      313              
==========================================
  Hits         1569     1569              
+ Misses       1027     1026       -1     
  Partials      230      230              

Files with missing lines	Coverage Δ
...b/Components/Features/Issues/IssueDetailPage.razor	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mpaulosky]

✅ Ralph (Work Monitor): All test suites pass (unit, integration, bunit, architecture). The E2E failure is a pre-existing CI infrastructure issue (DB init failure, Redis SSL errors) unrelated to this PR's changes. Merging.