Skip to content

Commit 040aa22

Browse files
hyperpolymathhyperpolymath
committed
chore: M5 CI/Workflow Sweep - final synchronisation
1 parent dd0de9c commit 040aa22

28 files changed

Lines changed: 175 additions & 175 deletions

File tree

‎did-you-actually-do-that/src/claim_extractor.rs‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,35 +73,35 @@ impl ClaimExtractor {
7373
// Must contain a file extension or path separator to be a valid file path
7474
file_created_pattern: Regex::new(
7575
r#"(?i)(?:I\s+)?(?:created|made|added)\s+(?:the\s+)?file\s+[`"']?([^\s`"']+\.[a-zA-Z0-9]+)[`"']?"#
76-
).unwrap(),
76+
).expect("TODO: handle error"),
7777
file_wrote_pattern: Regex::new(
7878
r#"(?i)(?:I\s+)?(?:wrote|written)\s+(?:to\s+)?(?:the\s+)?file\s+[`"']?([^\s`"']+\.[a-zA-Z0-9]+)[`"']?"#
79-
).unwrap(),
79+
).expect("TODO: handle error"),
8080
file_modified_pattern: Regex::new(
8181
r#"(?i)(?:I\s+)?(?:modified|updated|edited|changed)\s+(?:the\s+)?(?:file\s+)?[`"']?([^\s`"']+\.[a-zA-Z0-9]+)[`"']?"#
82-
).unwrap(),
82+
).expect("TODO: handle error"),
8383
file_deleted_pattern: Regex::new(
8484
r#"(?i)(?:I\s+)?(?:deleted|removed)\s+(?:the\s+)?file\s+[`"']?([^\s`"']+\.[a-zA-Z0-9]+)[`"']?"#
85-
).unwrap(),
85+
).expect("TODO: handle error"),
8686
// Command execution
8787
command_ran_pattern: Regex::new(
8888
r#"(?i)(?:I\s+)?(?:ran|executed)\s+(?:the\s+)?command\s+[`"']([^`"']+)[`"']"#
89-
).unwrap(),
89+
).expect("TODO: handle error"),
9090
// Git operations - require hash to be present for commit detection
9191
git_commit_pattern: Regex::new(
9292
r#"(?i)(?:committed|commit)\s+(?:with\s+)?(?:hash\s+)?([a-f0-9]{7,40})"#
93-
).unwrap(),
93+
).expect("TODO: handle error"),
9494
git_push_pattern: Regex::new(
9595
r#"(?i)(?:I\s+)?pushed\s+(?:to\s+)?(?:the\s+)?branch\s+[`"']?([^\s`"']+)[`"']?"#
96-
).unwrap(),
96+
).expect("TODO: handle error"),
9797
// Git branch - require "branch" keyword to avoid false positives
9898
git_branch_pattern: Regex::new(
9999
r#"(?i)(?:I\s+)?(?:created|switched\s+to)\s+(?:the\s+)?branch\s+[`"']?([^\s`"']+)[`"']?"#
100-
).unwrap(),
100+
).expect("TODO: handle error"),
101101
// Directory operations - require "directory" or "folder" keyword
102102
directory_created_pattern: Regex::new(
103103
r#"(?i)(?:I\s+)?(?:created|made)\s+(?:the\s+)?(?:directory|folder|dir)\s+[`"']?([^\s`"']+)[`"']?"#
104-
).unwrap(),
104+
).expect("TODO: handle error"),
105105
}
106106
}
107107

‎did-you-actually-do-that/src/hooks.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -408,7 +408,7 @@ mod tests {
408408
"working_directory": "/home/user"
409409
}"#;
410410

411-
let input = parse_hook_input(json).unwrap();
411+
let input = parse_hook_input(json).expect("TODO: handle error");
412412
assert_eq!(input.event, HookEvent::PostToolExecution);
413413
assert_eq!(input.tool_name, Some("Write".to_string()));
414414
}

‎did-you-actually-do-that/src/main.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,7 @@ fn output_json(reports: &[VerificationReport]) {
281281
"unverifiable": reports.iter().filter(|r| r.overall_verdict == Verdict::Unverifiable).count()
282282
}
283283
});
284-
println!("{}", serde_json::to_string_pretty(&output).unwrap());
284+
println!("{}", serde_json::to_string_pretty(&output).expect("TODO: handle error"));
285285
}
286286

287287
/// Output reports as SARIF (Static Analysis Results Interchange Format)
@@ -372,7 +372,7 @@ fn output_sarif(reports: &[VerificationReport]) {
372372
}]
373373
});
374374

375-
println!("{}", serde_json::to_string_pretty(&sarif).unwrap());
375+
println!("{}", serde_json::to_string_pretty(&sarif).expect("TODO: handle error"));
376376
}
377377

378378
/// Output reports as JUnit XML (for CI systems)

‎did-you-actually-do-that/src/mcp_server.rs‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -564,7 +564,7 @@ mod tests {
564564
let response = server.handle_initialize(json!(1));
565565

566566
assert!(response.result.is_some());
567-
let result = response.result.unwrap();
567+
let result = response.result.expect("TODO: handle error");
568568
assert_eq!(result["protocolVersion"], MCP_VERSION);
569569
assert!(result["serverInfo"]["name"].is_string());
570570
}
@@ -575,11 +575,11 @@ mod tests {
575575
let response = server.handle_tools_list(json!(1));
576576

577577
assert!(response.result.is_some());
578-
let result = response.result.unwrap();
579-
let tools = result["tools"].as_array().unwrap();
578+
let result = response.result.expect("TODO: handle error");
579+
let tools = result["tools"].as_array().expect("TODO: handle error");
580580
assert_eq!(tools.len(), 4);
581581

582-
let tool_names: Vec<&str> = tools.iter().map(|t| t["name"].as_str().unwrap()).collect();
582+
let tool_names: Vec<&str> = tools.iter().map(|t| t["name"].as_str().expect("TODO: handle error")).collect();
583583
assert!(tool_names.contains(&"verify_claim"));
584584
assert!(tool_names.contains(&"quick_verify"));
585585
assert!(tool_names.contains(&"compute_hash"));
@@ -594,7 +594,7 @@ mod tests {
594594
}));
595595

596596
assert!(result.is_ok());
597-
let text = result.unwrap();
597+
let text = result.expect("TODO: handle error");
598598
assert!(text.contains("Refuted"));
599599
}
600600

@@ -613,7 +613,7 @@ mod tests {
613613
}));
614614

615615
assert!(result.is_ok());
616-
let text = result.unwrap();
616+
let text = result.expect("TODO: handle error");
617617
assert!(text.contains("Test claim"));
618618
assert!(text.contains("Refuted"));
619619
}

‎did-you-actually-do-that/src/port_protocol.rs‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -417,7 +417,7 @@ mod tests {
417417
let resp = PortResponse::err("test-2".to_string(), -1, "oops".to_string());
418418
assert!(resp.result.is_none());
419419
assert!(resp.error.is_some());
420-
assert_eq!(resp.error.unwrap().code, -1);
420+
assert_eq!(resp.error.expect("TODO: handle error").code, -1);
421421
}
422422

423423
#[test]
@@ -431,7 +431,7 @@ mod tests {
431431
};
432432
let response = handle_request(&verifier, &ensemble, &request);
433433
assert!(response.result.is_some());
434-
assert_eq!(response.result.unwrap()["pong"], true);
434+
assert_eq!(response.result.expect("TODO: handle error")["pong"], true);
435435
}
436436

437437
#[test]
@@ -445,7 +445,7 @@ mod tests {
445445
};
446446
let response = handle_request(&verifier, &ensemble, &request);
447447
assert!(response.error.is_some());
448-
assert_eq!(response.error.unwrap().code, -32601);
448+
assert_eq!(response.error.expect("TODO: handle error").code, -32601);
449449
}
450450

451451
#[test]
@@ -463,7 +463,7 @@ mod tests {
463463
};
464464
let response = handle_request(&verifier, &ensemble, &request);
465465
assert!(response.result.is_some());
466-
let result = response.result.unwrap();
466+
let result = response.result.expect("TODO: handle error");
467467
assert!(result["votes"].is_array());
468468
assert!(result["suggested_verdict"].is_string());
469469
}
@@ -472,10 +472,10 @@ mod tests {
472472
fn test_read_write_message_roundtrip() {
473473
let data = b"hello world";
474474
let mut buf = Vec::new();
475-
write_message(&mut buf, data).unwrap();
475+
write_message(&mut buf, data).expect("TODO: handle error");
476476

477477
let mut cursor = io::Cursor::new(buf);
478-
let result = read_message(&mut cursor).unwrap().unwrap();
478+
let result = read_message(&mut cursor).expect("TODO: handle error").expect("TODO: handle error");
479479
assert_eq!(result, data);
480480
}
481481
}

‎did-you-actually-do-that/src/slm/ensemble.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -297,8 +297,8 @@ mod tests {
297297
fn test_ensemble_result_serialization() {
298298
let ensemble = Ensemble::new();
299299
let result = ensemble.evaluate(&sample_context());
300-
let json = serde_json::to_string(&result).unwrap();
301-
let back: EnsembleResult = serde_json::from_str(&json).unwrap();
300+
let json = serde_json::to_string(&result).expect("TODO: handle error");
301+
let back: EnsembleResult = serde_json::from_str(&json).expect("TODO: handle error");
302302
assert_eq!(back.votes.len(), result.votes.len());
303303
assert_eq!(back.suggested_verdict, result.suggested_verdict);
304304
}

‎did-you-actually-do-that/src/slm/mod.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,8 @@ mod tests {
177177
reasoning: "Stub functions detected".to_string(),
178178
weight: 1.5,
179179
};
180-
let json = serde_json::to_string(&vote).unwrap();
181-
let back: Vote = serde_json::from_str(&json).unwrap();
180+
let json = serde_json::to_string(&vote).expect("TODO: handle error");
181+
let back: Vote = serde_json::from_str(&json).expect("TODO: handle error");
182182
assert_eq!(back.voter, "test-model");
183183
assert_eq!(back.decision, VoteDecision::Reject);
184184
assert_eq!(back.weight, 1.5);

‎did-you-actually-do-that/src/verifiers/attestation.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ mod tests {
393393
#[test]
394394
fn test_valid_attestation() {
395395
let (envelope_str, report_bytes) = build_test_attestation();
396-
let envelope: Envelope = serde_json::from_str(&envelope_str).unwrap();
396+
let envelope: Envelope = serde_json::from_str(&envelope_str).expect("TODO: handle error");
397397
let result = verify_envelope(&envelope, &report_bytes, None);
398398
assert_eq!(result.verdict, Verdict::Confirmed);
399399
assert!(result.structure_valid);
@@ -407,7 +407,7 @@ mod tests {
407407
#[test]
408408
fn test_tampered_report_detected() {
409409
let (envelope_str, _) = build_test_attestation();
410-
let envelope: Envelope = serde_json::from_str(&envelope_str).unwrap();
410+
let envelope: Envelope = serde_json::from_str(&envelope_str).expect("TODO: handle error");
411411
// Tamper: different report bytes
412412
let tampered = b"tampered report content";
413413
let result = verify_envelope(&envelope, tampered, None);

‎did-you-actually-do-that/src/verifiers/completeness.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -464,7 +464,7 @@ mod tests {
464464

465465
let stub_file = tmp.join("stub.rs");
466466
fs::write(&stub_file, "fn main() { todo!() }\nfn helper() { unimplemented!() }")
467-
.unwrap();
467+
.expect("TODO: handle error");
468468

469469
let claims = vec![make_claim(
470470
"Created stub.rs",

‎did-you-actually-do-that/src/verifiers/content_hash.rs‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,17 +51,17 @@ mod tests {
5151

5252
#[test]
5353
fn test_hash_matches() {
54-
let dir = TempDir::new().unwrap();
54+
let dir = TempDir::new().expect("TODO: handle error");
5555
let file_path = dir.path().join("test.txt");
5656
let content = b"hello world";
57-
fs::write(&file_path, content).unwrap();
57+
fs::write(&file_path, content).expect("TODO: handle error");
5858

5959
let mut hasher = Sha256::new();
6060
hasher.update(content);
6161
let hash = hex::encode(hasher.finalize());
6262

6363
let evidence = EvidenceSpec::FileWithHash {
64-
path: file_path.to_str().unwrap().to_string(),
64+
path: file_path.to_str().expect("TODO: handle error").to_string(),
6565
sha256: hash,
6666
};
6767
let result = check(&evidence);
@@ -70,12 +70,12 @@ mod tests {
7070

7171
#[test]
7272
fn test_hash_mismatch() {
73-
let dir = TempDir::new().unwrap();
73+
let dir = TempDir::new().expect("TODO: handle error");
7474
let file_path = dir.path().join("test.txt");
75-
fs::write(&file_path, "hello world").unwrap();
75+
fs::write(&file_path, "hello world").expect("TODO: handle error");
7676

7777
let evidence = EvidenceSpec::FileWithHash {
78-
path: file_path.to_str().unwrap().to_string(),
78+
path: file_path.to_str().expect("TODO: handle error").to_string(),
7979
sha256: "0000000000000000000000000000000000000000000000000000000000000000"
8080
.to_string(),
8181
};

0 commit comments

Comments
 (0)