Skip to content

Commit e75747f

Browse files
committed
fix(ci): fix remaining formatting and dialyzer issues
- Fix additional code formatting in mcp_public_test.exs (line breaking) - Add dialyzer ignore for pattern_match_cov warning in packages.ex (defensive normalize_meta/1 clause for runtime safety)
1 parent 71e669d commit e75747f

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

.dialyzer_ignore.exs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,8 @@
1515
# These are available at runtime but not during dialyzer analysis
1616
~r/lib\/mix\/tasks\/test\.e2e\.ex.*unknown_function/,
1717
~r/test\/support\/conn_case\.ex.*unknown_function/,
18-
~r/test\/support\/admin_conn_case\.ex.*unknown_function/
18+
~r/test\/support\/admin_conn_case\.ex.*unknown_function/,
19+
# Ignore pattern match coverage warning in packages.ex normalize_meta/1
20+
# This is a defensive clause for handling non-map meta values at runtime
21+
~r/lib\/hex_hub\/mcp\/tools\/packages\.ex.*pattern_match_cov/
1922
]

test/hex_hub_web/controllers/mcp_public_test.exs

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,11 @@ defmodule HexHubWeb.MCPPublicTest do
2222
describe "User Story 1: AI Client Queries Package Information" do
2323
setup do
2424
# Create test packages for querying
25-
package = create_package(%{name: "phoenix", meta: %{description: "Web framework for Elixir"}})
26-
_package2 = create_package(%{name: "ecto", meta: %{description: "Database wrapper for Elixir"}})
25+
package =
26+
create_package(%{name: "phoenix", meta: %{description: "Web framework for Elixir"}})
27+
28+
_package2 =
29+
create_package(%{name: "ecto", meta: %{description: "Database wrapper for Elixir"}})
2730

2831
%{package: package}
2932
end
@@ -206,12 +209,14 @@ defmodule HexHubWeb.MCPPublicTest do
206209
describe "User Story 3: AI Client Retrieves Package Dependencies" do
207210
setup do
208211
# Create packages with releases for dependency tests
209-
package = create_package(%{name: "phoenix_live_view", meta: %{description: "LiveView for Phoenix"}})
212+
package =
213+
create_package(%{name: "phoenix_live_view", meta: %{description: "LiveView for Phoenix"}})
210214

211215
# Create release with tarball
212216
tarball = create_test_tarball("phoenix_live_view", "1.0.0")
213217
meta = %{"app" => "phoenix_live_view", "description" => "LiveView"}
214218
requirements = %{"phoenix" => %{"requirement" => "~> 1.7"}}
219+
215220
{:ok, _release} =
216221
HexHub.Packages.create_release("phoenix_live_view", "1.0.0", meta, requirements, tarball)
217222

0 commit comments

Comments
 (0)