Skip to content

Commit 65ef94c

Browse files
style: apply mix format to existing files
1 parent 02f05b7 commit 65ef94c

54 files changed

Lines changed: 795 additions & 290 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

lib/sentinel_cp/auth/signing_key.ex

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,15 @@ defmodule SentinelCp.Auth.SigningKey do
2626
"""
2727
def create_changeset(key, attrs) do
2828
key
29-
|> cast(attrs, [:org_id, :key_id, :public_key, :private_key_encrypted, :algorithm, :active, :expires_at])
29+
|> cast(attrs, [
30+
:org_id,
31+
:key_id,
32+
:public_key,
33+
:private_key_encrypted,
34+
:algorithm,
35+
:active,
36+
:expires_at
37+
])
3038
|> validate_required([:org_id, :key_id, :public_key, :private_key_encrypted])
3139
|> unique_constraint(:key_id)
3240
|> foreign_key_constraint(:org_id)

lib/sentinel_cp/bundles/config_validation_rule.ex

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,16 @@ defmodule SentinelCp.Bundles.ConfigValidationRule do
5151
"""
5252
def create_changeset(rule, attrs) do
5353
rule
54-
|> cast(attrs, [:name, :description, :rule_type, :pattern, :config, :severity, :enabled, :project_id])
54+
|> cast(attrs, [
55+
:name,
56+
:description,
57+
:rule_type,
58+
:pattern,
59+
:config,
60+
:severity,
61+
:enabled,
62+
:project_id
63+
])
5564
|> validate_required([:name, :rule_type, :project_id])
5665
|> validate_length(:name, min: 1, max: 100)
5766
|> validate_inclusion(:rule_type, @rule_types)
@@ -94,6 +103,7 @@ defmodule SentinelCp.Bundles.ConfigValidationRule do
94103

95104
"max_size" ->
96105
config = get_field(changeset, :config) || %{}
106+
97107
if Map.has_key?(config, "max_bytes") do
98108
changeset
99109
else
@@ -102,6 +112,7 @@ defmodule SentinelCp.Bundles.ConfigValidationRule do
102112

103113
"json_schema" ->
104114
config = get_field(changeset, :config) || %{}
115+
105116
if Map.has_key?(config, "schema") do
106117
changeset
107118
else

lib/sentinel_cp/bundles/diff.ex

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ defmodule SentinelCp.Bundles.Diff do
101101
end
102102

103103
defp extract_manifest_files(nil), do: %{}
104+
104105
defp extract_manifest_files(manifest) when is_map(manifest) do
105106
case Map.get(manifest, "files") do
106107
files when is_map(files) -> files

lib/sentinel_cp/bundles/risk.ex

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@ defmodule SentinelCp.Bundles.Risk do
5252

5353
defp check_auth_policy_changed(reasons, new_config, prev_config) do
5454
new_auth = extract_blocks(new_config, ~r/(?:auth|authentication|authorization)\s*\{[^}]*\}/s)
55-
prev_auth = extract_blocks(prev_config, ~r/(?:auth|authentication|authorization)\s*\{[^}]*\}/s)
55+
56+
prev_auth =
57+
extract_blocks(prev_config, ~r/(?:auth|authentication|authorization)\s*\{[^}]*\}/s)
5658

5759
if new_auth != prev_auth and prev_auth != [] do
5860
["auth_policy_changed" | reasons]

lib/sentinel_cp/nodes.ex

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -506,6 +506,7 @@ defmodule SentinelCp.Nodes do
506506
case Rollouts.plan_rollout(rollout) do
507507
{:ok, _} ->
508508
require Logger
509+
509510
Logger.info("Auto-remediation rollout started",
510511
rollout_id: rollout.id,
511512
node_id: node.id,
@@ -514,6 +515,7 @@ defmodule SentinelCp.Nodes do
514515

515516
{:error, reason} ->
516517
require Logger
518+
517519
Logger.warning("Auto-remediation rollout failed to plan",
518520
rollout_id: rollout.id,
519521
reason: inspect(reason)
@@ -522,6 +524,7 @@ defmodule SentinelCp.Nodes do
522524

523525
{:error, reason} ->
524526
require Logger
527+
525528
Logger.warning("Auto-remediation rollout creation failed",
526529
node_id: node.id,
527530
reason: inspect(reason)

lib/sentinel_cp/notifications.ex

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -149,10 +149,11 @@ defmodule SentinelCp.Notifications do
149149
total_managed: drift_stats.total_managed,
150150
drifted: drift_stats.drifted,
151151
in_sync: drift_stats.in_sync,
152-
percentage: if(drift_stats.total_managed > 0,
153-
do: Float.round(drift_stats.drifted / drift_stats.total_managed * 100, 2),
154-
else: 0.0
155-
)
152+
percentage:
153+
if(drift_stats.total_managed > 0,
154+
do: Float.round(drift_stats.drifted / drift_stats.total_managed * 100, 2),
155+
else: 0.0
156+
)
156157
},
157158
threshold: %{
158159
percentage: Project.drift_alert_threshold(project),
@@ -237,7 +238,12 @@ defmodule SentinelCp.Notifications do
237238
:ok
238239

239240
{:error, reason} ->
240-
Logger.warning("Notification failed", url: url, event: payload[:event], error: inspect(reason))
241+
Logger.warning("Notification failed",
242+
url: url,
243+
event: payload[:event],
244+
error: inspect(reason)
245+
)
246+
241247
{:error, reason}
242248
end
243249
end

lib/sentinel_cp/rollouts.ex

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,18 @@ defmodule SentinelCp.Rollouts do
88

99
import Ecto.Query, warn: false
1010
alias SentinelCp.Repo
11-
alias SentinelCp.Rollouts.{Rollout, RolloutStep, NodeBundleStatus, RolloutApproval, RolloutTemplate, TickWorker, HealthCheckEndpoint, HealthChecker}
11+
12+
alias SentinelCp.Rollouts.{
13+
Rollout,
14+
RolloutStep,
15+
NodeBundleStatus,
16+
RolloutApproval,
17+
RolloutTemplate,
18+
TickWorker,
19+
HealthCheckEndpoint,
20+
HealthChecker
21+
}
22+
1223
alias SentinelCp.{Bundles, Nodes, Notifications, Orgs, Projects}
1324

1425
## Rollout Template CRUD
@@ -329,7 +340,9 @@ defmodule SentinelCp.Rollouts do
329340
|> Repo.update()
330341

331342
# Send notification
332-
maybe_send_notification(fn -> Notifications.notify_rollout_rejected(rollout, user, comment) end)
343+
maybe_send_notification(fn ->
344+
Notifications.notify_rollout_rejected(rollout, user, comment)
345+
end)
333346

334347
result
335348
end
@@ -483,7 +496,13 @@ defmodule SentinelCp.Rollouts do
483496
if node_ids == [] do
484497
{:error, :no_target_nodes}
485498
else
486-
batches = chunk_into_batches(node_ids, rollout.strategy, rollout.batch_size, rollout.batch_percentage)
499+
batches =
500+
chunk_into_batches(
501+
node_ids,
502+
rollout.strategy,
503+
rollout.batch_size,
504+
rollout.batch_percentage
505+
)
487506

488507
result =
489508
Repo.transaction(fn ->
@@ -1205,6 +1224,7 @@ defmodule SentinelCp.Rollouts do
12051224

12061225
defp broadcast_and_notify(rollout, old_state, new_state) do
12071226
broadcast_rollout_update(rollout)
1227+
12081228
maybe_send_notification(fn ->
12091229
Notifications.notify_rollout_state_change(rollout, old_state, new_state)
12101230
end)

lib/sentinel_cp/rollouts/scheduler_worker.ex

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,7 @@ defmodule SentinelCp.Rollouts.SchedulerWorker do
3737
)
3838
end
3939
else
40-
Logger.debug(
41-
"SchedulerWorker: rollout #{rollout.id} scheduled but awaiting approval"
42-
)
40+
Logger.debug("SchedulerWorker: rollout #{rollout.id} scheduled but awaiting approval")
4341
end
4442
end
4543

lib/sentinel_cp_web/components/core_components.ex

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -521,7 +521,11 @@ defmodule SentinelCpWeb.CoreComponents do
521521
def detail_header(assigns) do
522522
~H"""
523523
<div class="border-b border-base-300 pb-4 mb-6">
524-
<.link :if={@back_path} navigate={@back_path} class="text-sm text-base-content/50 hover:text-base-content mb-2 inline-block">
524+
<.link
525+
:if={@back_path}
526+
navigate={@back_path}
527+
class="text-sm text-base-content/50 hover:text-base-content mb-2 inline-block"
528+
>
525529
&larr; Back
526530
</.link>
527531
<div class="flex items-center justify-between gap-4">

lib/sentinel_cp_web/controllers/api/drift_controller.ex

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -223,12 +223,14 @@ defmodule SentinelCpWeb.Api.DriftController do
223223
defp maybe_add_status_filter(opts, _), do: [{:include_resolved, true} | opts]
224224

225225
defp parse_int(nil, default), do: default
226+
226227
defp parse_int(val, default) when is_binary(val) do
227228
case Integer.parse(val) do
228229
{n, _} -> n
229230
:error -> default
230231
end
231232
end
233+
232234
defp parse_int(val, _default) when is_integer(val), do: val
233235

234236
defp drift_event_to_json(event) do
@@ -314,23 +316,27 @@ defmodule SentinelCpWeb.Api.DriftController do
314316
end
315317

316318
defp csv_escape(nil), do: ""
319+
317320
defp csv_escape(val) when is_binary(val) do
318321
if String.contains?(val, [",", "\"", "\n"]) do
319322
"\"#{String.replace(val, "\"", "\"\"")}\""
320323
else
321324
val
322325
end
323326
end
327+
324328
defp csv_escape(val), do: to_string(val)
325329

326330
defp calculate_duration(%{detected_at: detected, resolved_at: resolved})
327331
when not is_nil(detected) and not is_nil(resolved) do
328332
DateTime.diff(resolved, detected)
329333
end
334+
330335
defp calculate_duration(_), do: nil
331336

332337
defp maybe_add_date_filter(opts, _key, nil), do: opts
333338
defp maybe_add_date_filter(opts, _key, ""), do: opts
339+
334340
defp maybe_add_date_filter(opts, key, value) do
335341
case DateTime.from_iso8601(value) do
336342
{:ok, dt, _} -> [{key, dt} | opts]
@@ -346,25 +352,31 @@ defmodule SentinelCpWeb.Api.DriftController do
346352

347353
defp filter_since(events, nil), do: events
348354
defp filter_since(events, ""), do: events
355+
349356
defp filter_since(events, since) do
350357
case DateTime.from_iso8601(since) do
351358
{:ok, dt, _} ->
352359
Enum.filter(events, fn e ->
353360
DateTime.compare(e.detected_at, dt) in [:gt, :eq]
354361
end)
355-
_ -> events
362+
363+
_ ->
364+
events
356365
end
357366
end
358367

359368
defp filter_until(events, nil), do: events
360369
defp filter_until(events, ""), do: events
370+
361371
defp filter_until(events, until_dt) do
362372
case DateTime.from_iso8601(until_dt) do
363373
{:ok, dt, _} ->
364374
Enum.filter(events, fn e ->
365375
DateTime.compare(e.detected_at, dt) in [:lt, :eq]
366376
end)
367-
_ -> events
377+
378+
_ ->
379+
events
368380
end
369381
end
370382

0 commit comments

Comments
 (0)