Skip to content

Commit 187bc70

Browse files
committed
chore: remove em-dashes from comments
1 parent a9c4c74 commit 187bc70

1 file changed

Lines changed: 19 additions & 19 deletions

File tree

crates/phantom-cli/src/commands/doctor.rs

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ pub fn run(fix: bool) -> Result<()> {
105105
if env_is_gitignored(&project_dir, &content) {
106106
check_pass(".env is in .gitignore");
107107
} else {
108-
check_warn(".env is NOT in .gitignore secrets could be committed!");
108+
check_warn(".env is NOT in .gitignore, secrets could be committed!");
109109
check_fix("Run: echo '.env' >> .gitignore");
110110
if fix {
111111
let mut c = content;
@@ -121,7 +121,7 @@ pub fn run(fix: bool) -> Result<()> {
121121
}
122122
}
123123
} else {
124-
check_warn("No .gitignore consider adding one");
124+
check_warn("No .gitignore, consider adding one");
125125
if fix {
126126
std::fs::write(
127127
&gitignore_path,
@@ -139,7 +139,7 @@ pub fn run(fix: bool) -> Result<()> {
139139
if example_path.exists() {
140140
check_pass(".env.example found (team onboarding ready)");
141141
} else {
142-
check_warn("No .env.example team onboarding may be difficult");
142+
check_warn("No .env.example, team onboarding may be difficult");
143143
check_fix("Run: phantom env");
144144
if fix && env_path.exists() {
145145
if let Ok(dotenv) = DotenvFile::parse_file(&env_path) {
@@ -181,13 +181,13 @@ pub fn run(fix: bool) -> Result<()> {
181181
.iter()
182182
.any(|v| v.as_str().is_some_and(|s| s.contains(".env")));
183183
if has_env_deny {
184-
check_warn(".env is in deny rules after phantom init, .env is safe to read");
184+
check_warn(".env is in deny rules, after phantom init, .env is safe to read");
185185
issues += 1;
186186
}
187187
}
188188
}
189189
} else {
190-
check_info("No Claude Code config run `phantom setup` for auto-mode");
190+
check_info("No Claude Code config, run `phantom setup` for auto-mode");
191191
}
192192

193193
// Check 7: Cloud auth
@@ -196,7 +196,7 @@ pub fn run(fix: bool) -> Result<()> {
196196
check_pass("Cloud: logged in (token stored in keychain)");
197197
}
198198
None => {
199-
check_info("Cloud: not logged in run `phantom login` for cloud sync");
199+
check_info("Cloud: not logged in, run `phantom login` for cloud sync");
200200
}
201201
}
202202

@@ -248,7 +248,7 @@ pub fn run(fix: bool) -> Result<()> {
248248
issues += 1;
249249
}
250250
} else {
251-
check_info("Not a git repo pre-commit hook not applicable");
251+
check_info("Not a git repo, pre-commit hook not applicable");
252252
}
253253

254254
// Check 9: README mentions Phantom
@@ -279,7 +279,7 @@ pub fn run(fix: bool) -> Result<()> {
279279
check_info(&format!("Install source: {label}"));
280280
}
281281

282-
// Check 11: Vault backend (informational also shown inline above when
282+
// Check 11: Vault backend (informational, also shown inline above when
283283
// config exists, but surfaced unconditionally here so it's always visible)
284284
{
285285
if let Ok(config) = PhantomConfig::load(&config_path) {
@@ -303,20 +303,20 @@ pub fn run(fix: bool) -> Result<()> {
303303
format!("{bytes} B")
304304
};
305305
check_info(&format!(
306-
"Audit log: enabled {} ({})",
306+
"Audit log: enabled, {} ({})",
307307
path.display(),
308308
size_str
309309
));
310310
}
311311
Err(_) => {
312312
check_info(&format!(
313-
"Audit log: enabled {} (not yet created)",
313+
"Audit log: enabled, {} (not yet created)",
314314
path.display()
315315
));
316316
}
317317
},
318318
Err(_) => {
319-
check_info("Audit log: enabled (log path unresolvable HOME not set?)");
319+
check_info("Audit log: enabled (log path unresolvable, HOME not set?)");
320320
}
321321
}
322322
} else {
@@ -340,20 +340,20 @@ pub fn run(fix: bool) -> Result<()> {
340340
println!();
341341
println!(" {} MCP client wiring:", "info".blue());
342342

343-
// Claude Code project-local .claude/settings.local.json
343+
// Claude Code, project-local .claude/settings.local.json
344344
let claude_path = project_dir.join(".claude/settings.local.json");
345345
check_mcp_client("claude", &claude_path, false);
346346

347347
if let Some(home) = dirs::home_dir() {
348-
// Cursor ~/.cursor/mcp.json
348+
// Cursor, ~/.cursor/mcp.json
349349
check_mcp_client("cursor", &home.join(".cursor/mcp.json"), true);
350-
// Windsurf ~/.codeium/windsurf/mcp_config.json
350+
// Windsurf, ~/.codeium/windsurf/mcp_config.json
351351
check_mcp_client(
352352
"windsurf",
353353
&home.join(".codeium/windsurf/mcp_config.json"),
354354
true,
355355
);
356-
// Codex ~/.codex/config.toml
356+
// Codex, ~/.codex/config.toml
357357
check_mcp_client("codex", &home.join(".codex/config.toml"), true);
358358
}
359359
}
@@ -370,7 +370,7 @@ pub fn run(fix: bool) -> Result<()> {
370370
"!".yellow().bold(),
371371
issues,
372372
if !fix {
373-
" run `phantom doctor --fix` to auto-fix"
373+
", run `phantom doctor --fix` to auto-fix"
374374
} else {
375375
""
376376
}
@@ -525,7 +525,7 @@ mod tests {
525525

526526
#[test]
527527
fn env_covered_by_wildcard_glob_is_detected() {
528-
// `*.env` is the wildcard variant the issue calls out git check-ignore
528+
// `*.env` is the wildcard variant the issue calls out, git check-ignore
529529
// treats it as a match for `.env`, and our fallback scan does too.
530530
let tmp = tempfile::tempdir().unwrap();
531531
init_git_repo(tmp.path());
@@ -545,13 +545,13 @@ mod tests {
545545

546546
#[test]
547547
fn comment_lines_do_not_count_as_a_match() {
548-
// Pure-text fallback path (no git repo) comments must not satisfy the check.
548+
// Pure-text fallback path (no git repo), comments must not satisfy the check.
549549
let tmp = tempfile::tempdir().unwrap();
550550
let content = "# .env\nnode_modules/\n";
551551
assert!(!env_is_gitignored(tmp.path(), content));
552552
}
553553

554-
/// Smoke test detect_install_source() must be stable across two calls.
554+
/// Smoke test, detect_install_source() must be stable across two calls.
555555
#[test]
556556
fn install_source_is_stable() {
557557
let a = detect_install_source();

0 commit comments

Comments
 (0)