Conversation
|
เรียน @wasintoh อ.โต ช่วยรีวิวด้วยคร๊าาบบบ จาก Codex User ครับ |
|
ขอบคุณมากครับ 🙏 เรื่องที่คุณเจอถูกทั้งหมดครับ AGENTS.md ของเราใหญ่ 117,191 bytes ผมต้องขอโทษด้วย v2.1 ที่แก้เรื่องเดียวกันสร้างเสร็จแล้วแต่ผมยังไม่ได้ v2.1 ทำอะไรบ้าง: AGENTS.md เหลือ 12,949 bytes พร้อม assertion มีจุดหนึ่งอยากขอความเห็นครับ โค้ดคุณเขียน skills ลง .codex/skills/ https://developers.openai.com/codex/skills ไม่มี .codex/skills อยู่ในรายการเลย ถ้าเป็นตามนั้น ไฟล์จะถูกเขียนลง อีกเหตุผลที่ผมเลือก .agents/skills คือ Codex, Cursor 2.4 และ ผมเลยยังไม่ merge PR นี้นะครับ เพราะฐานมันไม่มีแล้ว และ uninstall.js แต่ผมไม่ปิดครับ เพราะของที่มีค่าที่สุดใน PR นี้ไม่ใช่การแก้ Codex ขอเวลาไม่กี่วันให้ผม push v2.1 ขึ้น main ก่อนนะครับ แล้วรบกวนเปิด จะกลับมาคอมเมนต์ตรงนี้ทันทีที่ v2.1 ขึ้น main ครับ ขอบคุณอีกครั้งครับ 🙏 |
…ce uninstall process - Changed installation paths from .codex/skills to .agents/skills for better organization. - Updated uninstall logic to handle new paths and added dry-run functionality. - Enhanced AGENTS.md handling to ensure size limits are enforced and backups are created. - Modified tests to reflect changes in directory structure and functionality. - Improved documentation to clarify the new structure and usage.
|
เรียน @wasintoh อ.โต ปรับแก้ไขตาม comment มาแล้วเรียบร้อยครับ 😊 |
|
อัพเดทตามที่รับปากไว้ครับ v2.1.0 ขึ้น main แล้ว และ publish ขึ้น npm เรียบร้อย npx toh-framework@latest installเรื่องที่คุณจุดขึ้นมากับที่ v2.1 แก้คือเรื่องเดียวกันเป๊ะครับ ตอนนี้ AGENTS.md ที่ generate ออกมาอยู่ที่ 12,959 bytes มี assertion ที่ทำให้ install ล้มทันทีถ้าไฟล์โตเกิน 24 KiB (กันพลาดซ้ำถาวร) และเขียน skills ไปอยู่ที่ แล้วมีของแถมที่ไม่ได้ตั้งใจ ระหว่างเทสผมลองกับ ZCode (Z.ai) ปรากฏว่ามันอ่าน path ชุดเดียวกันเป๊ะ เลยได้รองรับเพิ่มอีกตัวโดยแทบไม่ต้องเขียนอะไรใหม่ ตอนนี้ ขอบคุณมากที่เป็นคนนอกคนแรกที่ลองแล้วบอกนะครับ ถึง PR จะไม่ได้ merge ตรงๆ เพราะฐานมันเป็นก่อน v2.1 ไปแล้ว แต่ประเด็นที่คุณเห็นถูกแก้ครบทุกข้อ ยังค้างใจอยู่ข้อเดียวจากรอบก่อนครับ ถ้าคุณมีโอกาสลองอีกที รบกวนช่วยดูให้หน่อยว่าใน session Codex จริงๆ มันโหลด skill จาก |
|
version 2.1 ออกแล้วนะครับ แก้เยอะมากๆ คุณ @pcbimon ทดสอบได้เลยนะครับ ขอบคุณมากๆครับ |
🧠 ปัญหา
การทำงานร่วมกับ Codex เดิมของ TOH จำลอง คำสั่ง
/toh-*ผ่านAGENTS.mdขนาด ~900 บรรทัด (ฝัง agent bodies ทั้งหมด + ตาราง "Command Recognition")
แต่ Codex ไม่มีระบบ slash command แบบกำหนดเอง — การทำงานจึงพึ่งการ
"หลอก" โมเดลผ่าน prompt ซึ่งไม่เสถียร และไม่มีอะไรที่ Codex discover ได้จริง
นอกจากนี้ reinstall แต่ละครั้งยังเขียนทับ
.toh/memory/*(state ที่ผู้ใช้ใช้งานอยู่)🔍 สาเหตุ (Root Cause)
Integration เดิมเกิดก่อนที่ Codex จะมี native skills
(
.codex/skills/<name>/SKILL.mdที่ Codex scan จาก project root,เรียกใช้ด้วย
$skill-nameหรือ/skillsหรือจับคู่อัตโนมัติจาก description)— ตรวจสอบกับ source code ของ
openai/codexแล้วว่ารองรับ path นี้จริง✨ วิธีแก้
generate จาก frontmatter ของ
src/commands/*.md(single source of truth)— wrapper เป็น thin layer ที่อ้างถึง workflow จริงใน
.toh/commands/และ supporting skills ใน
.toh/skills/ไม่มีการก๊อปปี้เนื้อหาซ้ำAGENTS.mdเหลือ managed block สั้นๆ (71 บรรทัด): identity,capabilities, ตาราง skills, หมายเหตุ compatibility สำหรับ
/toh-*และ memory protocol
sequential · ไม่มี Stop hook → บังคับตัวเองไม่จบจน plan ครบ · ไม่มี
model routing → ข้าม tier ต่างๆ
toh uninstall(--ide codexลบเฉพาะไฟล์ที่ TOH สร้าง —skill ของผู้ใช้และข้อความใน AGENTS.md อยู่ครบ) และใช้ teardown เดียวกัน
ในเส้นทาง Fresh Install
--quickไม่ interactive สมบูรณ์(เดิม crash/hang เมื่อรัน headless) และ
.toh/memory/*ถูก seedเฉพาะเมื่อยังไม่มีไฟล์ (ตรงกับที่ README สัญญาไว้)
🔄 Compatibility
(ยกเว้นเลขเวอร์ชัน 2.0.0 → 2.1.0) — ตรวจด้วย baseline diff จาก
git archive HEADเหมือนเดิม · state ใน
.toh/ไม่ถูกแตะ/toh-planแบบพิมพ์ข้อความยังใช้ได้ — AGENTS.md สั่งให้ Codex ตีความเป็นการเรียก skill ที่ตรงกัน (ระบุชัดว่าเป็น compatibility behavior ไม่ใช่
native command)
✅ Tests
tests/codex.test.js(node:test) 10 เคส: fresh install,เก็บข้อความเดิมใน AGENTS.md, idempotency/determinism, ไม่แตะ skill
ของผู้ใช้, ลบ stale TOH skill, uninstall แบบ scoped, ความถูกต้องของ
SKILL.md (frontmatter + reference ที่ resolve ได้จริง)
npm testและ CI/release gate (Node 18 + 22)ไม่มี drift เลยแม้แต่ไฟล์เดียว · uninstall ในโปรเจคที่มี skill ของ
ผู้ใช้เอง → ลบเฉพาะ 14 TOH skills + TOH block ใน AGENTS.md
session เดียว (ระบุไว้ในทุก skill + AGENTS.md)