Repository navigation
chore(explorer): clean up leftovers in the docs page, runner and API - #41
Merged
Merged
Conversation
… OpenAPI spec The FastAPI app was built with a hardcoded version "0.1.0", which Swagger and any spec consumer showed. It now takes the installed agentenv-framework version through openapi_docs.package_version (public now that app.py uses it). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The explorer always serves its spec live, so the hosted hub's published-spec path never runs: the Commit / Bucket / Latest key / Version key rows, the "Spec commit" lines in the header and sidebar, and the metadata fields only they read. The header's "Version" line goes too; the This Hub rows already show the framework and protocol versions. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Nothing passes onEditEvaluator or onRerunFromStep, and handleEvaluatorSaved was only stored in a ref nothing read, so the flow and the context it captured never ran: pendingEvalRun and its effect, lastContextJson / lastContextSource, the savedContextJson / onContextCaptured props, the poll's latestCompleted, and EVALUATOR_STEP_TYPES. The app shell's write-only editTaskId, cloneTaskId, evaluatorOnlyTaskId and rerunFromStep state goes with it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… saves trigger state The UI README still said its docs would land with the OSS release; it now covers developing against agent-env up, building the static export it serves, and links the user guide. prompt_agent's trigger-state docstring said S3; it writes to the configured object store. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…t's served content type
GET /{collection}/{id}/versions answered 200 [] for an id with no versions while GET /{id} 404s; it now 404s too, and a page past the last version of a stored entity is still []. The UI's two callers already treat a non-OK response as no versions. GET /objects/metadata returned content_type null for stores that keep none; it now reports the type /objects/content serves, from the shared fallback to the filename. Tests cover both and the OpenAPI version from the first commit.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
npm run lint ran next lint, which prompted for setup and is deprecated in Next 15.5. It now runs eslint with a flat config extending next/core-web-vitals and next/typescript (eslint 9, eslint-config-next 15.5.25, matching next). The three errors it found were unescaped apostrophes in JSX; 61 warnings remain and do not fail the run. The CI ui job runs the lint after the typecheck. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
4 tasks done
earakely-scale
approved these changes
Oct 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cleanup of explorer leftovers found during the OSS pass. One commit per item.
agentenv-frameworkversion instead of a hardcoded0.1.0.onEditEvaluator/onRerunFromStep).ui/README.md; theprompt_agentdocstring says object store, not S3./{id}/versionsis now a 404 for an unknown id, likeGET /{id}. Both UI callers already handle a 404./objects/metadatareports the same content type/objects/contentserves.npm run lintruns ESLint 9 witheslint-config-next(no longer the deprecatednext lint). It found 3 errors, now fixed, and CI'suijob now runs lint. 61 warnings are deferred (mostly unused vars).Testing
test_mcp_cli_builderfailures that also fail on main. No plugin-API break.agent-env up: the Docs page shows the real versions with no Bucket rows, the runner runs, and universe pages load.🤖 Generated with Claude Code
The PR appears safe to merge; the changes since the previous review introduce no actionable issue.
Summary
The explorer now reports live package and object details more consistently, and its Docs page shows only metadata for the live hub. The PR also removes an unused task-runner rerun path and updates the UI lint and setup docs.
Reviews (2) · Last reviewed commit: "docs(explorer): trim the versions and co..."