mirror of
https://github.com/crewAIInc/crewAI.git
synced 2026-09-24 11:50:16 +00:00
* feat(tracing): record the last traced run for crewai eval instead of printing it After a traced run, crewAI printed a panel with the execution id and a viewer link. The id is internal, and the panel exposed it for no purpose a user has. Nothing is printed now. Instead, once the run's spans have reached Wharf, crewAI writes `.crewai/last_run.json` in the project: the execution id, when the run started and ended, whether it was traced anonymously or under an account, and the AMP base url. `crewai eval` reads it back, so the user evaluates their last run without pasting anything. GrantSpanExporter.record_export replaces show_trace_summary at the two finish points (an authenticated run's shutdown, an anonymous run's share). Only a run whose every export succeeded is recorded — a partial export would name a run the grader could not read whole. Recording is silent in the TUI and under message suppression too, off under the test suite, and inert inside a deployment, where the platform binds the execution before crewAI's own tracing starts. The record is written atomically and a write failure never fails the run. The crew, flow and json_crew scaffolds now ignore `.crewai/` like the declarative flow scaffold already did. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * fix(tracing): never record a truncated ephemeral run; resolve the record path inside the guard; pin the window and the once - An ephemeral buffer that dropped spans at its cap still uploaded the rest and recorded the run. The grader could not read that run whole, so it is no longer recorded (the upload still happens). Test added. - last_run_path() ran before the try in record_last_run; a cwd that vanished mid-run would have raised out of a function documented never to raise. It is inside now, with the same debug log and None. - Tests: the "recorded once" check counted an unchanged mtime, which proves nothing on a coarse filesystem — it now counts calls to record_last_run; the first-start/last-end fold across export batches had no value test — batches now arrive out of order and the ISO window is asserted. - Docs: the conversational-flows comment said "one trace link" for output that no longer exists; tracing.mdx now names .crewai/last_run.json, what it holds, who reads it, and when it is not written. en, ar, ko, pt-BR. 85 passed (test_session_trace_export + test_last_run); ruff and mypy clean. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * fix(tracing): a temporary file of its own per writer; the docs name deployments - record_last_run wrote every record through one `.tmp` name, so two crews finishing together in one project could replace each other's temporary file and one record was lost. Each write now goes through tempfile.mkstemp in the same directory, then os.replace; a failed write removes its own temporary file. Tests: two writers use two names and leave only last_run.json; a failed replace leaves nothing behind. - tracing.mdx (en, ar, ko, pt-BR): nothing is recorded inside a deployment, where the platform owns the trace. 87 passed (test_last_run + test_session_trace_export); ruff and mypy clean. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * fix(tracing): the run that finished last stays recorded, whichever writer comes last Two crews finishing together in one project: the older run's writer could reach the file after the newer one and leave the older run as "the last". record_last_run now compares finished_at (recorded_at when absent) with the record already there and keeps the later-finished run; the same run recorded again (a refreshed grant) and a record without a comparable time never block the run just finished. No lock file: the file is a convenience pointer and the compare closes the ordering, leaving a microsecond window between read and replace that two runs finishing in the same instant could hit — either is then a fair "last run". Test: the older writer replaces after the newer one; the newer stays. 88 passed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * fix(tracing): a same-millisecond tie keeps the record already there; nothing is recorded inside a deployment - The times in the record are stored to the millisecond; two runs finishing in the same millisecond compared equal and the later writer won, which could leave the older run recorded. A tie now keeps what is there. No finer order is worth a field in the file. - recording_enabled() is false when the platform's integration token is present: a deployment normally never reaches this code (the host binds the trace before crewAI would start its own), but a container that did not would have written a file the platform never reads, against what the docs promise. Test added. 89 passed; ruff and mypy clean. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * fix(tracing): only a run that provably finished later holds the record My previous commit compared `finished_at` and fell back to `recorded_at`, which broke two existing tests on CI: two records written in the same millisecond with no completion time compared equal, so the second write was dropped and the older run stayed. Locally the two writes happened to land in different milliseconds, so the suite passed — a timing-dependent bug, not a CI quirk. The rule is now narrow: a write stands unless the record already there is a DIFFERENT run with a later `finished_at`. No completion time on either side, the same run recorded again, or a tie to the millisecond all leave the write to stand, so "the last run recorded" stays what a reader gets. The case the rule exists for is unchanged: the older-finished writer arriving last does not replace the newer one. 485 passed in lib/crewai/tests/telemetry (test_last_run run five times for timing); ruff and mypy clean. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix(tracing): serialise the write across processes; drop the platform-token guard, which was not a deployment test - Reading the record, deciding and replacing it are now one step, held under a lock file beside the record, so a writer can no longer decide against a record another process has already replaced. Where the platform has no flock the write goes ahead unserialised: the record is a convenience pointer for `crewai eval`, never worth failing a run over. The regression drives three writers at once and fails without the lock. - The deployment guard I added yesterday read CREWAI_PLATFORM_INTEGRATION_TOKEN, which is not a deployment marker: `crewai create crew` writes that token into a project's own .env for platform tools, and crew_base loads it, so the guard would have stopped recording for every developer who uses them. It is also unnecessary — a deployment kicks off with tracing off, so crewAI never builds the exporter and never reaches this module. Guard removed, with a test that a project carrying that token is still recorded. - The module docstring claimed the platform "binds the execution before crewAI would start its own tracing". It does not; it sets tracing off at kickoff. Docstring and the four tracing docs now say the real reason. 487 passed in lib/crewai/tests/telemetry; ruff and mypy clean. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix(tracing): a refused lock still records the run; the platform-token test exercises the real guard Two Cursor findings on the locking commit. - If flock itself failed — some network mounts refuse it — the OSError escaped _exclusive, the outer handler deleted the temporary file and record_last_run returned None, so the run was not recorded at all. A refused lock is now logged and the write goes ahead unserialised, matching what already happened when the lock file could not be opened or flock does not exist. Unserialised beats not recorded. - test_a_project_using_platform_tools_is_still_recorded used the `project` fixture, which stubs recording_enabled — the very thing under test — so the regression it guards could have come back green. It now stubs only project_dir and asserts the real guard. 13 in test_last_run, 489 across lib/crewai/tests/telemetry. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
213 lines
10 KiB
Python
213 lines
10 KiB
Python
"""`.crewai/last_run.json`: the run `crewai eval` evaluates by default."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import json
|
|
import threading
|
|
import time
|
|
|
|
import pytest
|
|
|
|
from crewai.telemetry.tracing import last_run
|
|
|
|
|
|
def _leftovers(project):
|
|
"""Anything in .crewai that is neither the record nor its lock file."""
|
|
directory = project / ".crewai"
|
|
keep = {last_run.LAST_RUN_FILE, last_run.LOCK_FILE}
|
|
return [child.name for child in directory.iterdir() if child.name not in keep] if directory.exists() else []
|
|
|
|
|
|
@pytest.fixture
|
|
def project(monkeypatch, tmp_path):
|
|
monkeypatch.setattr(last_run, "project_dir", lambda: tmp_path)
|
|
monkeypatch.setattr(last_run, "recording_enabled", lambda: True)
|
|
return tmp_path
|
|
|
|
|
|
def test_a_run_is_recorded_atomically_and_read_back(project):
|
|
path = last_run.record_last_run(
|
|
execution_id="6f31fe1a-20bd-4bfe-a011-25d6b9341f62",
|
|
tier="ephemeral",
|
|
started_at_ns=1_758_240_000_000_000_000,
|
|
finished_at_ns=1_758_240_009_500_000_000,
|
|
amp_base_url="https://app.crewai.com",
|
|
)
|
|
assert path == project / ".crewai" / "last_run.json"
|
|
assert not _leftovers(project) # no temporary file left behind
|
|
written = json.loads(path.read_text(encoding="utf-8"))
|
|
assert written["execution_id"] == "6f31fe1a-20bd-4bfe-a011-25d6b9341f62"
|
|
assert written["tier"] == "ephemeral"
|
|
assert written["started_at"] == "2025-09-19T00:00:00.000+00:00"
|
|
assert written["finished_at"] == "2025-09-19T00:00:09.500+00:00"
|
|
assert written["amp_base_url"] == "https://app.crewai.com"
|
|
assert written["recorded_at"]
|
|
assert last_run.read_last_run(project) == written
|
|
|
|
|
|
def test_the_newest_run_replaces_the_previous_one(project):
|
|
last_run.record_last_run(execution_id="first", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
last_run.record_last_run(execution_id="second", tier="authenticated", started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
written = last_run.read_last_run(project)
|
|
assert written is not None and written["execution_id"] == "second"
|
|
assert written["started_at"] is None and written["finished_at"] is None
|
|
|
|
|
|
def test_nothing_is_recorded_under_the_test_suite(monkeypatch, tmp_path):
|
|
monkeypatch.setattr(last_run, "project_dir", lambda: tmp_path)
|
|
monkeypatch.setenv("CREWAI_TESTING", "true")
|
|
assert last_run.record_last_run(execution_id="x", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None) is None
|
|
assert not (tmp_path / ".crewai").exists()
|
|
|
|
|
|
def test_a_project_using_platform_tools_is_still_recorded(monkeypatch, tmp_path):
|
|
"""`crewai create crew` writes CREWAI_PLATFORM_INTEGRATION_TOKEN into the project's
|
|
own .env, so it says "this developer uses platform tools", never "this is a
|
|
deployment". Reading it as a deployment marker would leave those users with no
|
|
run for `crewai eval` to find.
|
|
|
|
Deliberately NOT the `project` fixture: that one stubs `recording_enabled`, which
|
|
is the very thing under test here."""
|
|
monkeypatch.setattr(last_run, "project_dir", lambda: tmp_path)
|
|
monkeypatch.delenv("CREWAI_TESTING", raising=False)
|
|
monkeypatch.setenv("CREWAI_PLATFORM_INTEGRATION_TOKEN", "a token from the project's .env")
|
|
|
|
assert last_run.recording_enabled() is True # the real guard, not the fixture's stub
|
|
path = last_run.record_last_run(execution_id="local", tier="authenticated", started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
|
|
assert path is not None
|
|
assert last_run.read_last_run(tmp_path)["execution_id"] == "local"
|
|
|
|
|
|
def test_a_missing_or_broken_record_reads_as_none(project):
|
|
assert last_run.read_last_run(project) is None
|
|
(project / ".crewai").mkdir()
|
|
(project / ".crewai" / "last_run.json").write_text("not json", encoding="utf-8")
|
|
assert last_run.read_last_run(project) is None
|
|
(project / ".crewai" / "last_run.json").write_text(json.dumps({"tier": "ephemeral"}), encoding="utf-8")
|
|
assert last_run.read_last_run(project) is None # no execution id: no run
|
|
|
|
|
|
def test_a_write_failure_never_raises(project, monkeypatch):
|
|
(project / ".crewai").write_text("a file where the directory should be", encoding="utf-8")
|
|
assert last_run.record_last_run(execution_id="x", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None) is None
|
|
|
|
|
|
def test_each_writer_uses_a_temporary_file_of_its_own(project, monkeypatch):
|
|
"""Two crews finishing together in one project: neither may replace the other's temporary file."""
|
|
replaced: list[str] = []
|
|
real_replace = last_run.os.replace
|
|
monkeypatch.setattr(last_run.os, "replace", lambda src, dst: replaced.append(str(src)) or real_replace(src, dst))
|
|
for execution_id in ("first", "second"):
|
|
last_run.record_last_run(execution_id=execution_id, tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
assert len(replaced) == 2 and replaced[0] != replaced[1]
|
|
names = [source.rsplit("/", 1)[-1] for source in replaced]
|
|
assert all(name.startswith(".last_run.json.") and name.endswith(".tmp") for name in names)
|
|
assert not _leftovers(project)
|
|
|
|
|
|
def test_a_failed_write_leaves_no_temporary_file(project, monkeypatch):
|
|
monkeypatch.setattr(last_run.os, "replace", lambda src, dst: (_ for _ in ()).throw(OSError("disk full")))
|
|
assert last_run.record_last_run(execution_id="x", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None) is None
|
|
assert not _leftovers(project) and not (project / ".crewai" / last_run.LAST_RUN_FILE).exists()
|
|
|
|
|
|
def test_the_run_that_finished_last_stays_recorded_whichever_writer_comes_last(project):
|
|
"""Two crews finish together; the OLDER run's writer gets to the file after the newer one did."""
|
|
second = 1_000_000_000
|
|
base = 1_758_240_000 * second
|
|
last_run.record_last_run(execution_id="newer", tier="ephemeral", started_at_ns=base, finished_at_ns=base + 30 * second, amp_base_url=None)
|
|
kept = last_run.record_last_run(execution_id="older", tier="ephemeral", started_at_ns=base, finished_at_ns=base + 10 * second, amp_base_url=None)
|
|
written = last_run.read_last_run(project)
|
|
assert kept == project / ".crewai" / "last_run.json"
|
|
assert written is not None and written["execution_id"] == "newer"
|
|
assert not _leftovers(project) # the loser's temporary file is gone
|
|
|
|
# The same run recorded again (a refreshed grant) and a run that finished later both replace it.
|
|
last_run.record_last_run(execution_id="newer", tier="authenticated", started_at_ns=base, finished_at_ns=base + 30 * second, amp_base_url=None)
|
|
assert last_run.read_last_run(project)["tier"] == "authenticated"
|
|
last_run.record_last_run(execution_id="newest", tier="ephemeral", started_at_ns=base, finished_at_ns=base + 40 * second, amp_base_url=None)
|
|
assert last_run.read_last_run(project)["execution_id"] == "newest"
|
|
|
|
# A tie to the millisecond: either run is a fair "last run", so the write stands.
|
|
last_run.record_last_run(execution_id="same-instant", tier="ephemeral", started_at_ns=base, finished_at_ns=base + 40 * second + 400_000, amp_base_url=None)
|
|
assert last_run.read_last_run(project)["execution_id"] == "same-instant"
|
|
|
|
# An OLDER run still loses that tie-free comparison, however late its writer arrives.
|
|
last_run.record_last_run(execution_id="stale", tier="ephemeral", started_at_ns=base, finished_at_ns=base + 5 * second, amp_base_url=None)
|
|
assert last_run.read_last_run(project)["execution_id"] == "same-instant"
|
|
|
|
# A record without a comparable time never blocks the run just finished.
|
|
(project / ".crewai" / "last_run.json").write_text(json.dumps({"execution_id": "legacy"}), encoding="utf-8")
|
|
last_run.record_last_run(execution_id="fresh", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
assert last_run.read_last_run(project)["execution_id"] == "fresh"
|
|
|
|
|
|
def test_writers_are_serialised_so_a_stale_read_cannot_overwrite_a_newer_run(project, monkeypatch):
|
|
"""Two crews finish at once. Reading the file, deciding and replacing it are one
|
|
step, so no writer can decide against a record another writer has already replaced."""
|
|
second = 1_000_000_000
|
|
base = 1_758_240_000 * second
|
|
depth = 0
|
|
overlaps = []
|
|
guard = threading.Lock()
|
|
real_keep = last_run._keep
|
|
|
|
def slow_keep(record, existing):
|
|
nonlocal depth
|
|
with guard:
|
|
depth += 1
|
|
if depth > 1:
|
|
overlaps.append(depth)
|
|
time.sleep(0.05) # the window a stale read would live in
|
|
try:
|
|
return real_keep(record, existing)
|
|
finally:
|
|
with guard:
|
|
depth -= 1
|
|
|
|
monkeypatch.setattr(last_run, "_keep", slow_keep)
|
|
runs = [("oldest", 10), ("newest", 40), ("middle", 20)]
|
|
writers = [
|
|
threading.Thread(
|
|
target=last_run.record_last_run,
|
|
kwargs={"execution_id": name, "tier": "ephemeral", "started_at_ns": base,
|
|
"finished_at_ns": base + offset * second, "amp_base_url": None},
|
|
name=name,
|
|
)
|
|
for name, offset in runs
|
|
]
|
|
for writer in writers:
|
|
writer.start()
|
|
for writer in writers:
|
|
writer.join(10)
|
|
|
|
assert overlaps == [] # never two writers inside the read-decide-replace region
|
|
written = last_run.read_last_run(project)
|
|
assert written is not None and written["execution_id"] == "newest"
|
|
assert not _leftovers(project)
|
|
|
|
|
|
def test_a_write_still_happens_where_the_platform_has_no_file_locking(project, monkeypatch):
|
|
"""Windows has no flock: the record is a convenience pointer, never worth failing a run over."""
|
|
monkeypatch.setattr(last_run, "fcntl", None)
|
|
|
|
path = last_run.record_last_run(execution_id="unlocked", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
|
|
assert path is not None
|
|
assert last_run.read_last_run(project)["execution_id"] == "unlocked"
|
|
assert not (project / ".crewai" / last_run.LOCK_FILE).exists()
|
|
|
|
|
|
@pytest.mark.parametrize("failure", [OSError(45, "Operation not supported"), PermissionError(13, "Permission denied")])
|
|
def test_a_filesystem_that_refuses_the_lock_still_records_the_run(project, monkeypatch, failure):
|
|
"""Some network mounts have no working flock. Unserialised beats not recorded."""
|
|
monkeypatch.setattr(
|
|
last_run.fcntl, "flock", lambda handle, operation: (_ for _ in ()).throw(failure)
|
|
)
|
|
|
|
path = last_run.record_last_run(execution_id="unlockable", tier=None, started_at_ns=None, finished_at_ns=None, amp_base_url=None)
|
|
|
|
assert path is not None
|
|
assert last_run.read_last_run(project)["execution_id"] == "unlockable"
|