Add ECOSYSTEM_CARDS entry for pmll skill.md - #99
Conversation
|
🤖 Automated message from Kaan's Automated Triage Bot. 👀 Picked this up — a review will follow shortly. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the site’s ecosystem skills data by shortening several card descriptions and adding a new “PMLL” ecosystem card.
Changes:
- Replaced multiple full
descriptionstrings with truncated text containing[...]. - Minor formatting change to
copyValuefor one card. - Added a new ecosystem card entry for “PMLL”.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bd49fd5cdb
ℹ️ 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".
Adding the PMLL entry also rewrote 16 ECOSYSTEM_CARDS descriptions as copies truncated mid-sentence with a literal "[...]". These strings are card data, not display markup: they render verbatim on the card grid, feed the community-search haystack, and are written into public/llms.txt. Truncating them drops real capability terms from search and shows agents and crawlers placeholder text. Restore every description to its full value and keep the new PMLL entry, leaving the diff as a single added card.
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for this — and genuinely for flagging the dead link yourself in review rather than waiting to be chased for it. The card entry is well-formed: all four fields match Two things before it can go on the catalog. The skill needs to be public. I pushed a commit restoring the truncated descriptions. The branch had rewritten 16 I skipped Copilot's other suggestion, a Ping me when the SKILL.md is live and I'll pick this back up. |
|
I saw that! Yeah copilot needed to be skipped! Thank you you so much! Let me make sure to skill.md is live! |
|
https://github.com/drQedwards/pmll Is now live from template |
|
@kaankacar ping! It is live! |
|
🤖 Automated message from Kaan's Automated Triage Bot. That did it — What I can't confirm is the Stellar part, and on this catalog that's the load-bearing claim. SKILL.md leads with memory "on Stellar" and "mapped to Stellar accounts", but across the whole public repo SKILL.md is the only file that mentions Stellar or Soroban at all — no Three smaller things in the quick start, all checkable:
Could you point me at where the Stellar integration lives? If it's planned rather than shipped, that's a completely fine answer — it just means the card and SKILL.md should describe what the skill does today. Since "does this belong on the Stellar catalog" is a listing call rather than mine to make alone, I'm handing that part to @kaankacar and @ElliotFriend. |
|
Now I'll add the clarifying comment to the PR mentioning @kaankacar and @ElliotFriend:Here are all the comments on the PR: PR #99 Comments SummaryCopilot Bot Reviews (Early automated checks)Multiple automated review comments flagged truncated descriptions:
Codex Bot (Community Search Impact)Codex Bot (Dead Link Warning)Your Reply (drQedwards)Long response outlining options:
You concluded: "Best practice: don't merge until public SKILL.md is reachable" Your Follow-up CommentYou linked: Current Status✅ SKILL.md now fixed (commit
❌ Still pending: Stellar integration clarification for @kaankacar and @ElliotFriend Now I need to add a comment tagging them: … again! @kaankacar and @ElliotFriend this human in the loop signals and pings one last time! |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for turning this around so fast. I checked all three quick-start fixes in Answering the Stellar question by removing the claims from SKILL.md rather than stretching to fit was the right instinct, and I'd much rather have that answer than an invented integration. That does leave one loose end: the card description in this PR still reads "persistent spatial and hyperdimensional memory on Stellar" — the exact phrase your own SKILL.md just dropped. Whichever way the listing goes, the card and the file it links to shouldn't disagree, so could you update this PR's The bigger question — whether a general-purpose agent-memory MCP belongs on the Stellar skills catalog — is a listing call rather than mine to make, and it's already with @kaankacar and @ElliotFriend on the hold I opened last round. I've added today's findings there. Nothing else is blocking from my side. |
|
So the hyperdimensional hallucination is why grok can handle a sdk mid context for the terminal, but cannot automate a PR pitch in public for the DAO. but yeah…. The connectors aren’t even real. Very very ghostly those connector plugins. No invention at all. And that right there I’m saying as Josef, the human being here. Oh btw, MCP servers are for the normies. Simple actionable link into the catalogue. https://github.com/supermodeltools/cli already reduces token burn by 50% , (my primitives, such as peek(); and the kv slotting in the cache are very much real vibes that I assure you this will long the stellar catalogue project thus yes… you get the double sword where the fees in gas aren’t as high: YOU AVOID INFLATION in stellar. This solution already has in mind and scope the 192k issues for the 666 repos surrounding the protocol @kaankacar @ElliotFriend for drip, which i asked as a question relating to socketfi.app for the discord admin who did the town hall.! whether or not? (Tangent!$ Okay, and Ezz is having a space surrounding interchaining dynamics with the boilerplate. (He’s talking about Robinhood and autoscout. https://x.com/ezz_wins/status/2088395675301986627?s=46)! Right now I can merge everyone into this forever todo, but your catalogue (code name: Raven)!🐦⬛ will only shimmer with the stellar shine of mere persistence. You want to talk shop about quantum and hyperdimensional embeddings? Before the 16th? Let’s see a merge of this first before wave 45 ends and we enter 46 with this first. I promise to clean away the hallucination for QA work. My fellowship at joinhandshake taught well on that end with superannotate.
|
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for the follow-up — and to be straight with you, since you asked for a merge before the 16th: I can't make that call, and it isn't a timing problem. Whether a general-purpose agent-memory MCP belongs on the Stellar catalog is with @kaankacar and @ElliotFriend on the hold I opened, and this stays blocked until one of them closes it. Everything else on my side is done: the card is well-formed, the diff is still just your one entry, and last round's SKILL.md corrections all re-check clean. One small thing is still outstanding on this branch — the I did check the token-burn argument, since it was offered as the Stellar connection. supermodeltools/cli is real, but the savings it advertises are agent/LLM token costs ("40%+" by its own README), and those don't touch Stellar transaction fees — a base fee in stroops plus Soroban resource metering, not gas. The inflation operation is also deprecated (per the docs glossary, it "wasn't working as intended"), so there's no inflation there to avoid. That doesn't move the listing question either way; I just didn't want to leave it unanswered. |
|
Well either way you know I have read enough of the documentation to ask a question: @kaankacar if you were to get a slice of the pie with Elliot, what is your one patch code recommendation fix? No contradiction on the stellar context here, but what is the clean “template” push for me to intergrate for stellar? Ownership as a non-profit?! I want to give this where WE own it now. To decentralize correctly? $150,000 , 5 devs in this. Please, I beg you, what is the push you want me to lead into so I can plan for the next 30-90 days showcasing a roadmap and having the energy to go up and do the town hall for this? thank you. |
|
🤖 Automated message from Kaan's Automated Triage Bot. I'll answer the half I actually can. How to structure ownership, whether a non-profit is the right vehicle, and where five devs and a budget should point over the next 90 days — that's genuinely @kaankacar's and @ElliotFriend's to answer, not mine, and I'd rather say so than improvise something that sounds official because a bot said it. The technical half I can answer, and it's narrower than you might expect. The gap here has never been effort or quality — it's that no code path in PMLL touches Stellar. The smallest real version is one memory operation that writes a commitment to contract storage and reads it back through One working path like that is what would let the card say "on Stellar" and let me verify it rather than take it on faith. Nothing has moved on either branch since we last spoke (PR still |
|
To find the path? To show I can do the path finding? Excellent! The breadcrumbs I appreciate where I need this to securely touch the boilplate without exposing anything with tiny impact of 64 kib. I can do that. Give me 30 minutes. |
|
Yes.
One atomic payload (the 32-byte commitment) is sufficient, cheaper, and exactly the shape the triage bot asked for. No distributor automator or multi-step SocketFi-style guardian flow is required for the minimal verifiable path that unblocks PR #99.
[lib] [dependencies] [dev-dependencies] [profile.release] [profile.release-with-logs] /// Compact key space. Using a pure 32-byte ID keeps the serialized #[contract] #[contractimpl] } #[cfg(test)] } stellar.toml for PMLL – Persistent Memory Layer for Stellar AgentsVERSION = "2.0.0" NETWORK_PASSPHRASE = "Test SDF Network ; September 2015" # change to PUBLIC for mainnet [[CURRENCIES]] Contract registry – the only on-chain surface needed for the skill card[[CONTRACTS]] [DOCUMENTATION] [[PRINCIPALS]] [HORIZON] optional – leave blank if you only use RPC
I got the syntax errors cleaned up for the actual commit now iterated |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for this — the design is right, and that was the part I couldn't hand you. A 32-byte commitment on-chain with the payload off-chain is exactly the shape that works, and your key layout respects both caps: To be straight about where things stand: nothing has landed yet.
You can skip the Two things I can't do from here. The listing question is still with @kaankacar and @ElliotFriend on the open hold, so I won't merge — and I'd rather you knew that before spending the weekend on the contract than after. And this card's |
|
Excuse me, can someone concisely explain the relevance of this "next-generation Python package manager" plus "persistent memory logic loop" to projects building on Stellar? This PR discussion reads like a generative brainstorming session 🤨 |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks — you pushed the contract within minutes of that last review, the shape is right (32-byte commitment on-chain, payload off-chain), and the The claim moved further than the code did, though. Four of last round's five are still open, and the first is the one that matters:
@aolieman — fair question. Short answer: as it stands there's no verifiable connection. PMLL is a well-built agent-memory MCP; the Stellar piece is a contract committed today, neither deployed nor called by the memory code. The listing call itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging. |
|
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for moving on this so fast — and for cutting "hyperdimensional" from SKILL.md in I do have to flag the deploy, because it's the one claim here that reads as already verified. I don't read that as deliberate — a plausible-looking Otherwise things are where we left them, so I won't relitigate: nothing under The unblocking step is genuinely small: deploy it for real and paste the deploy transaction hash, and I'll confirm it from here with one The listing question itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging either way — that isn't a no, and today's findings are on the issue. |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for the link — I did open it, but I don't think it shows what I need, and I'd rather ask than guess. The one outstanding ask is unchanged, and it's small. Nothing has moved on either repo since we last spoke (PR still The listing call itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging. That hasn't changed, and it isn't a no. |
SummaryAdds / updates the PMLL ecosystem card so it accurately reflects the current state of the project. Current reality
What this PR does
Follow-up (after merge)
This should clear the previous triage hold caused by mismatched claims. cc @kaankacar @ElliotFriend — ready for re-review / clearance of the hold when you have a moment. Prefer squash-merge. Test plan
|
Update fixes the issue with the agent handing out C… |
|
🤖 Automated message from Kaan's Automated Triage Bot. The PR description you just posted is the most accurate version of this yet — "on-chain commitment anchoring is planned, not yet live" is exactly the right framing, and it picks up the The trouble is that neither artifact matches it. This PR is also still at So the ask is the same shape as before but smaller: push the card wording from your own PR body, and re-land the SKILL.md corrections on the new repo. Then the card, the file, and the description all say the same true thing. (Small process note — this repo merges with merge commits, not squash.) The listing call itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging either way. Today's findings are on the issue. |
Add ppm to the .ts and the path for "https://github.com/drQedwards/ppm/blob/main/SKILL.md Alongside pmll
Update skills.ts
|
🤖 Automated message from Kaan's Automated Triage Bot. The SKILL.md re-land is the honest version, and I want to say so plainly: it's back to "on-chain commitment anchoring on Stellar … is planned", "off-chain today", no "hyperdimensional", and Three things on this push, though. The new SKILL.md is truncated. It ends at 1,122 bytes inside an unclosed The card wording is still the old one. This PR's PMLL Otherwise the diff is clean: 14 insertions, no deletions, all four fields match The listing call itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging either way — that isn't a no. Today's findings are on the issue. |
|
@kaankacar @ElliotFriend the old card has been updated.! PR #100 is for moving from planning into execution with a x402 skill known as MPP.md (jokingly called deployer.md for what it does with this skill). |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks for the ping — I went looking for the updated card and I think I've found the mix-up. #100 ("Patch 1 of PR #99") doesn't contain it: it adds Two things from last round are also unchanged, and I re-checked both rather than assuming. Credit where it's due, though: SKILL.md reads honestly now — anchoring described as planned, "off-chain today", and The listing call itself is still with @kaankacar and @ElliotFriend on the open hold, so I'm not merging either way — that isn't a no. Today's findings are on the issue. |

Here’s the content cleaned and properly formatted as Markdown:
PMLL Stellar Deploy + PR #99 Update
Deploy Commands (run on a machine with stellar-cli)
Clean Update for PR #99
1. Updated card entry
Replace the existing entry in
site/src/data/skills.ts:2. Updated
SKILL.mdFull replacement for
https://github.com/drQedwards/pmll/blob/main/SKILL.md:DrQEdwards/pmll just got fully updated as well.
mcp/directory in this repo for concrete commands and the 15 available tools (short-term KV, long-term graph, solution engine, etc.).Usage examples
// as an aside, the hallucination loop is coming from the fact that Grok is so sandboxed in a hypdimension sphere. That word only exists here in this PR. It’s been taken care of after triage flagged it.
Planned Stellar integration
A minimal Soroban contract (
contracts/pmll-anchor) is prepared to:pmll/anchorevent for indexers.Once deployed and called, the card description and this file will be updated to reflect live on-chain anchoring.
Links
License
MIT