Repository navigation
docs: fix agent guide gaps from the evaluations - #86
Conversation
The Track B and Track C agent evaluations found answers and code that the guide did not prevent: - A Codex attempt installed 0.14.2 from memory and read that old documentation. The install step now says to take the current release. - Two answers said Rails 8.0+. The gem description now states Ruby 3.3 and Rails 7.1, and a test asserts it. - Agent code called reject with one argument, used id inside an actor, and called schedule without an operation. The guide now shows the two arguments of reject and lists these mistakes with the correct form. A Ruby actor defines actor_id and no id, which was checked by loading the gem. - The install step now points to step 5, because the generated policies deny every call.
 |
Track C finished: 13 of 16 implementation attempts passed. Its report proposed three more changes, applied here: - Codex opened the agent guide in 1 of 8 attempts. The README now names the agent guide at the start of Installation. - One attempt computed expiry from the clock in a query, so a hold read as released before the reminder ran. The agent guide now says that a reminder changes state only when it runs and that a query must read the committed state. - The current-release note now also appears where Track C pointed.
|
@greptileai Please review the latest commit 42ac757. It adds the Track C findings: the README names the agent guide at the start of Installation, the agent guide says a reminder changes state only when it runs under solid_objects start, and the quickstart repeats the current-release note. rake quickstart still passes. |
|
Docs7 for cardmagic/solid-objects-ruby
Commit |
|
@greptileai Please review the latest commit 2c49cb8. It prepares 0.17.2: version.rb, Gemfile.lock, and the dated CHANGELOG section. No other change. |
1 participant
Why
The Track B (installed skill) and Track C (implementation) agent evaluations found answers and code that
docs/agents.mddid not prevent.What changes
test/unit/gem_specification_test.rbasserts it.rejectwith one argument, usedidinside an actor, and calledschedulewithout an operation. The guide showsreject(:room_full, "The room is full")and adds a table of these mistakes with the correct form. Loading the gem confirms that an actor definesactor_idand noid.Validation
Expected /requires Ruby 3\.3 or newer and Rails 7\.1 or newer/ to match ...) and passes after it.bundle exec rake: 873 runs, 0 failures, 28 database skips; Standard, RuboCop, RBS, Steep, and Brakeman are clean. One earlier run failedPostCommitTest#test_caller_cleanup_preserves_a_callback_record_lookup_error_and_its_original_causewith a 5 secondSyncTimeoutwhile the machine was loaded; this branch changes no runtime code, and the file passed 3 of 3 times alone and in the next full run.