Update examples, remove reference to skills

Author: Sushant20

01-tutorials/01-AgentCore-runtime/12-coding-agents/03-codex-with-s3-files-openai-api-key/README.md

@@ -30,13 +30,13 @@ Deploys Codex CLI  as an HTTP agent on AWS Bedrock AgentCore Runtime, with an S3
                             │  (agentcore-<account-id>)    │
                             │                              │
                             │  agents/                     │
-                            │  ├── skills/                 │
+                            │  ├── scripts/                │
                             │  ├── results/                │
                             │  └── ...                     │
                             └──────────────────────────────┘
 ```
 
-Multiple runtime sessions mount the same S3 Files file system, enabling agents to share skills, results, and data across independent invocations.
+Multiple runtime sessions mount the same S3 Files file system, enabling agents to share scripts, results, and data across independent invocations.
 
 ```
 CloudFormation stack (cfn-vpc.yaml):
@@ -146,24 +146,31 @@ python update.py
 
 ### Step 3 — Invoke the agent
 
-Send a prompt to the deployed agent. The first call creates a new session; subsequent calls can reuse the session ID for conversation continuity.
+Send a prompt to the deployed agent. The first call creates a new session; subsequent calls can reuse the session ID and Codex thread ID for conversation continuity.
 
-**Session A** — create a shared skill on the persistent filesystem:
+**Session A** — create a shared resource on the persistent filesystem:
 
 ```bash
-python invoke.py "can u create a new skill, to review python code? This skill should be created into /mnt/s3files/skills/"
+python invoke.py "Write a file at /mnt/s3files/scripts/python_review.md with a Python code review checklist. Use the shell to create the directory and write the file directly."
 ```
 
 Continue the conversation within the same session:
 
 ```bash
-python invoke.py --session <session-a-id> "now add unit tests for that skill"
+python invoke.py --session <session-id> --codex-session <codex-thread-id> "can you review python code, what standards are you going to use?"
 ```
 
-**Session B** — a completely new session accesses the same filesystem and uses the skill created by Session A:
+View the standards defined in the persistent file:
 
 ```bash
-python invoke.py "list the skills available in /mnt/s3files/skills/ and use the python review skill to review this code: def add(a,b): return a+b"
+python invoke.py --session <session-id> --codex-session <codex-thread-id> "what is inside /mnt/s3files/scripts/python_review.md, read it"
+```
+
+**Session B** — a completely new session accesses the same filesystem:
+
+```bash
+# New session automatically sees the same /mnt/s3files content
+python invoke.py "list the files in /mnt/s3files/scripts/ and show me the python_review.md checklist"
 ```
 
 Both sessions share `/mnt/s3files`, so anything written by one session is immediately available to others.

01-tutorials/01-AgentCore-runtime/12-coding-agents/03-codex-with-s3-files-openai-api-key/invoke.py

@@ -50,7 +50,7 @@ def load_config() -> dict:
         sys.exit(1)
 
 
-def invoke(runtime_arn: str, prompt: str, region: str, session_id: str = None) -> dict:
+def invoke(runtime_arn: str, prompt: str, region: str, session_id: str = None, codex_session_id: str = None) -> dict:
     client = boto3.client(
         "bedrock-agentcore",
         region_name=region,
@@ -61,6 +61,8 @@ def invoke(runtime_arn: str, prompt: str, region: str, session_id: str = None) -
         session_id = str(uuid.uuid4())
 
     payload_data = {"prompt": prompt}
+    if codex_session_id:
+        payload_data["sessionId"] = codex_session_id
 
     try:
         response = client.invoke_agent_runtime(
@@ -84,6 +86,7 @@ def invoke(runtime_arn: str, prompt: str, region: str, session_id: str = None) -
     print(f"  Status:          {response.get('statusCode', 'N/A')}")
 
     body["_runtimeSessionId"] = runtime_session
+    body["_codexSessionId"] = body.get("sessionId")
     return body
 
 
@@ -94,12 +97,18 @@ def main():
 
     args = sys.argv[1:]
     session_id = None
+    codex_session_id = None
 
     if "--session" in args:
         idx = args.index("--session")
         session_id = args[idx + 1]
         args = args[:idx] + args[idx + 2:]
 
+    if "--codex-session" in args:
+        idx = args.index("--codex-session")
+        codex_session_id = args[idx + 1]
+        args = args[:idx] + args[idx + 2:]
+
     if args:
         prompts = [" ".join(args)]
     else:
@@ -111,19 +120,23 @@ def main():
     print(f"Invoking agent: {runtime_arn}")
     if session_id:
         print(f"Resuming session: {session_id}")
+    if codex_session_id:
+        print(f"Resuming Codex thread: {codex_session_id}")
     print()
 
     for prompt in prompts:
         print(f"--- Prompt: {prompt}")
-        result = invoke(runtime_arn, prompt, region, session_id)
+        result = invoke(runtime_arn, prompt, region, session_id, codex_session_id)
         print(f"--- Response:\n{result.get('response', result)}")
         session_id = result.get("_runtimeSessionId", session_id)
-        print(f"--- Session ID: {session_id}")
+        codex_session_id = result.get("_codexSessionId", codex_session_id)
+        print(f"--- Session ID:       {session_id}")
+        print(f"--- Codex thread ID:  {codex_session_id}")
         print()
 
-    if session_id:
+    if session_id and codex_session_id:
         print("To continue this conversation:")
-        print(f"  python invoke.py --session {session_id} \"your next prompt\"")
+        print(f"  python invoke.py --session {session_id} --codex-session {codex_session_id} \"your next prompt\"")
 
 
 if __name__ == "__main__":

01-tutorials/01-AgentCore-runtime/12-coding-agents/03-codex-with-s3-files-openai-api-key/server.js

@@ -7,7 +7,7 @@ function runCodex(prompt, sessionId) {
   return new Promise((resolve, reject) => {
     let args;
     if (sessionId) {
-      args = ["exec", "resume", sessionId, prompt, "--json", "--sandbox", "danger-full-access", "--skip-git-repo-check"];
+      args = ["exec", "resume", "--json", "-c", 'sandbox_mode="danger-full-access"', sessionId, prompt];
     } else {
       args = ["exec", prompt, "--json", "--sandbox", "danger-full-access", "--skip-git-repo-check"];
     }
@@ -29,22 +29,36 @@ function runCodex(prompt, sessionId) {
     proc.stderr.on("data", (d) => (stderr += d));
 
     proc.on("close", (code) => {
-      console.log(`[runCodex] exited code=${code} stderr="${stderr}" stdout="${stdout.slice(0, 200)}"`);
+      console.log(`[runCodex] exited code=${code}`);
+      console.log(`[runCodex] stderr: ${stderr}`);
+      console.log(`[runCodex] stdout: ${stdout}`);
       if (code !== 0) {
         reject(new Error(`codex exited ${code}: ${stderr}`));
         return;
       }
-      // --json emits one JSON object per line, last one has the result
+      // --json emits one JSON object per line
+      // thread_id is in the "thread.started" line; response text in "item.completed"
       const lines = stdout.trim().split("\n").filter(Boolean);
-      try {
-        const last = JSON.parse(lines[lines.length - 1]);
-        resolve({
-          response: last.result || last.message || stdout.trim(),
-          sessionId: last.session_id || null,
-        });
-      } catch {
-        resolve({ response: stdout.trim(), sessionId: null });
+      let threadId = null;
+      let responseText = null;
+      for (const line of lines) {
+        try {
+          const obj = JSON.parse(line);
+          if (obj.type === "thread.started" && obj.thread_id) {
+            threadId = obj.thread_id;
+          }
+          if (obj.type === "item.completed" && obj.item && obj.item.text) {
+            responseText = obj.item.text;
+          }
+        } catch {
+          // skip non-JSON lines
+        }
       }
+      console.log(`[runCodex] thread_id=${threadId || "(none)"}`);
+      resolve({
+        response: responseText || stdout.trim(),
+        sessionId: threadId,
+      });
     });
     proc.on("error", reject);
   });