test: refine smoke judge comparison rules and output

- Focus comparison on matching event types to reduce false negatives
- Drop "ignore callProgress" rule (we're eliding them from the event
  stream before sending them to the judge now)

Signed-off-by: Nick Hale <[email protected]>

Author: njhale

pkg/tests/judge/judge.go

@@ -40,6 +40,8 @@ After making a determination, respond with a JSON object that conforms to the fo
   ]
 }
 
+If you determine actual and expected are not equivalent, include a diff of the parts of actual and expected that are not equivalent in the reasoning field of your response.
+
 Your responses are concise and include only the json object described above.
 `
 

pkg/tests/smoke/smoke_test.go

@@ -82,8 +82,8 @@ func TestSmoke(t *testing.T) {
 				expectedEvents,
 				actualEvents,
 				`
-- disregard differences in timestamps, generated IDs, natural language verbiage, and event order
-- omit callProgress events from the comparison
+- disregard differences in event order, timestamps, generated IDs, and natural language verbiage, grammar, and punctuation
+- compare events with matching event types
 - the overall stream of events and set of tools called should roughly match
 - arguments passed in tool calls should be roughly the same
 - the final callFinish event should be semantically similar