Add AGIJobManager Prime Mainnet Console HTML (2026-04-02)#218
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 10cbcf7537
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <div class="alphaTokenMeta"> | ||
| <div class="smallNote">TokenURI name</div> | ||
| <div style="font-weight:700" id="alphaIdentityTokenMetaName">Not applicable before issuance</div> | ||
| <div class="smallNote" id="alphaIdentityTokenMe |
There was a problem hiding this comment.
Escape preview image URI before assigning innerHTML
The cover preview builds an <img> tag with src interpolated directly from jobImageURI/metadata without escaping. A crafted value containing quotes can break out of the attribute and inject scriptable HTML (for example via copied/imported metadata values), which will execute when preview renders. This should use DOM APIs (createElement + img.src = ...) or escape attribute content before inserting HTML.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 122e042590
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <div class="smallNote">TokenURI name</div> | ||
| <div style="font-weight:700" id="alphaIdentityTokenMetaName">Not applicable before issuance</div> | ||
| <div class="smallNote" id="alphaIdentityTokenMe |
There was a problem hiding this comment.
Include owner/moderator getters in AGIJobManager ABI
refreshAdminPanels reads agiJobManager.methods.owner() and agiJobManager.methods.moderators(userAccount) to compute authorization posture, but AGIJobManagerABI does not declare those functions, so these calls always resolve to safe-read errors and executeAdminControl treats the wallet as unauthorized for manager admin actions. In practice this makes the manager admin controls non-operational even for the real owner/moderator.
Useful? React with 👍 / 👎.
| <div class="smallNote">TokenURI name</div> | ||
| <div style="font-weight:700" id="alphaIdentityTokenMetaName">Not applicable before issuance</div> | ||
| <div class="smallNote" id="alphaIdentityTokenMe |
There was a problem hiding this comment.
Add missing ENSJobPages read methods to the ABI
updateEnsJobPagePreview invokes ensContract.methods.owner()/ens()/nameWrapper()/jobLabelPrefix()/jobLabelSnapshot(...), but those getters are absent from ENSJobPagesABI; missing methods throw before .call().catch(...) can run, so the preview path falls into the outer catch and clears ENS state. This also prevents reliable ENS ownership posture in admin flows because the same undeclared reads are reused there.
Useful? React with 👍 / 👎.
Motivation
Description
ui/agijobmanager_genesis_job_mainnet_2026-04-02.html, a complete SPA-like HTML file with styles, responsive mobile dock/sheet, and app UI for Premium Discovery, Create Job, Jobs table, Operator Deck, Admin panels, Alpha-Agent identity, and bridge/mint flows.v26dock/sheet), v35 visual refinements, and extensive local-state persistence, packet tooling, and inspector/debug panels; add helper functions for IPFS/ENS normalization and completion inspection.createPremiumJobWithDiscovery), attach path, participate (commit/reveal) flows, and vault conversion for bridged → official AGIALPHA minting.Testing
Codex Task