Skip to content

feat: enhance branding and layout for ship deployment and status bar#61

Merged
WhyAsh5114 merged 18 commits into
mainfrom
feat/51-keeperhub-ship-ui
May 3, 2026
Merged

feat: enhance branding and layout for ship deployment and status bar#61
WhyAsh5114 merged 18 commits into
mainfrom
feat/51-keeperhub-ship-ui

Conversation

@jaykerkar0405

@jaykerkar0405 jaykerkar0405 commented May 3, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates both the product documentation (README.md) and integration feedback (FEEDBACK.md) for Crucible's integration with KeeperHub. The changes clarify the architecture and workflow of Crucible's agentic development environment, especially how it interacts with KeeperHub for on-chain automation, and provide detailed feedback and suggestions for KeeperHub's API and developer experience.

Key changes include:

Documentation and Product Narrative Updates

  • Updated the README to clarify that KeeperHub is used for post-deploy automation (not contract deployment itself), emphasizing the agent's workflow for wiring keepers and on-chain actions via KeeperHub's hosted MCP and tools. The documentation now reflects that deployments can be local or to 0G Galileo, and KeeperHub is used for automation after deployment. [1] [2] [3] [4] [5]

Integration Feedback and Suggestions for KeeperHub

  • Revised the feedback file to provide more actionable and precise suggestions for KeeperHub, including:
    • Clarifying authentication: Recommend documenting that server-side agents should use Bearer tokens for the hosted MCP, not OAuth, and suggest adding code snippets for popular SDKs.
    • Improving documentation: Point out issues with the network field (should be a stringified chainId, not a name or int), the need to cross-link wallet integration discovery for walletId, and the lack of contract creation actions in the action-schema.
    • Feature requests: Suggest exposing output schemas for actions, and clarifying which actions support contract creation (or explicitly stating if it's out of scope).

Thematic Grouping of Feedback

  • The feedback is now grouped by theme (UX friction, documentation gaps, feature requests), with concrete suggestions and clear severity ratings for each issue, making it easier for KeeperHub to act on the recommendations.

Consistency and Clarity

  • Updated terminology and descriptions throughout the documentation to consistently reflect Crucible's architecture (e.g., "KeeperHub automation MCP" instead of "Ship to public chains"), and clarified the division of responsibilities between Crucible (deployment) and KeeperHub (automation). [1] [2] [3] [4] [5] [6]

@sundaram123krishnan sundaram123krishnan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

WhyAsh5114 added 6 commits May 3, 2026 16:18
The original integration attempted to deploy contracts by setting
contractAddress: 0x000...0 in a KeeperHub Workflow node, reverse-engineering
an undocumented schema. KeeperHub does not support contract creation as an
action type, so this approach never worked.

Removes:
- backend ship API (ship.ts, route mount in index.ts)
- mcp-deployer KeeperHub REST client (keeperhub-client.ts)
- Ship* event types from agent-events (ShipSimulated, ShipStatus, ShipConfirmed)
- Frontend ship row components and inspector ship section
Replaces the hand-rolled REST shim with a direct connection to KeeperHub's
official hosted MCP server via @ai-sdk/mcp HTTP transport.

- backend/inference.ts: reads KEEPERHUB_API_KEY + KEEPERHUB_MCP_URL env vars
  and passes them through to runAgentTurn
- agent/loop.ts: per-turn createMCPClient with Bearer auth; tools namespaced
  keeperhub_* via toolToServer map; logs connected tool count on success,
  warns without crashing when unavailable
- agent/system-prompt.ts: Workflow C rewritten for KeeperHub post-deploy
  automation; Workflow B2 (Sepolia) added; deploy_sepolia in standard
  workflows; unverified-contract ABI guidance added
Adds a new deployer tool that deploys compiled contracts to Sepolia
(chainId 11155111) using a funded wallet — the same pattern as deploy_og_chain
but targeting Ethereum Sepolia instead of 0G Galileo.

Also strips the now-deleted ship/KeeperHub tools from the deployer server.

New env vars on the deployer node:
- SEPOLIA_DEPLOY_PRIVATE_KEY (required to enable the tool)
- SEPOLIA_RPC_URL (optional, defaults to https://rpc.ankr.com/eth_sepolia)

Returns address, txHash, gasUsed, chainId 11155111, and Etherscan Sepolia URL.
Deployment is recorded in the session registry under network 'sepolia'.
FEEDBACK.md: 5 grounded feedback items from actually using the hosted MCP
(Bearer vs OAuth friction, chainId string format, walletId discovery,
no CREATE action type, missing output schemas in list_action_schemas).
Adds 'What worked very well' section.

README.md, ARCHITECTURE.md, todo.md: remove Ship-to-Sepolia references;
position KeeperHub as the post-deploy automation layer. Update architecture
table and Phase 4 tracking.
@WhyAsh5114 WhyAsh5114 force-pushed the feat/51-keeperhub-ship-ui branch from cfeafc4 to dd444c8 Compare May 3, 2026 11:54
@WhyAsh5114 WhyAsh5114 merged commit c7fd2d5 into main May 3, 2026
5 checks passed
@WhyAsh5114 WhyAsh5114 deleted the feat/51-keeperhub-ship-ui branch May 3, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants