Skip to content

Tracer: Explicitly edit a received original and expose its current contents through message references #633

Closed

Description

@DoodleBears

Tracer: Explicitly edit a received original and expose its current contents through message references

Parent

#631 — PersonaBot attachment file operations.

What to build

Human asks the PersonaBot's Orchestrator to edit a specific received text attachment in place. Reuse the first file-operation slice's trusted attachment access, add explicitly requested original writeback, and let Human download or reopen the original message's Attachment to see the changed current contents. Intentional references sharing the Attachment identity see the same current file; independent uploads stay independent.

Acceptance criteria

  • Human can ask the real Orchestrator to read and explicitly edit one currently authorized original attachment using ordinary file capabilities. Default ZIP processing continues to use an independent saved working file.
  • Resolve exact source association and current access before original access/writeback; a supplied file ID or raw Host path cannot select an unrelated original. Leaving the source Channel blocks subsequent source operations.
  • Original writes update the owning profile-managed file. Original message reads/downloads and explicit shared-identity references return the changed current contents after refresh/restart.
  • Two independently uploaded identical files remain independent; editing one leaves the other unchanged. The Human uploader's source file and any remote platform original remain unchanged.
  • Ordinary current-file semantics apply. Add no application-level compare-before-write guard, lock, merge protocol or file-version archive; retain native Tool checks and existing Shell approval where used.
  • The file edit itself creates no Source Revision, file-change Inbox Admission or additional Bot wake. An explicit conversational reply remains an ordinary message action.
  • Missing files, revoked source access and write failures produce clear failure. No stale upload snapshot is used to recreate a missing original or claim write success.
  • Provide a Human-operable real-model edit and download verification, plus GitHub-rendered E2E screenshots or recording in the PR. Validate default working-copy behavior remains intact.
  • Update bilingual release guidance and architecture as needed, carry claim/commit/PR task trace, complete repository checks and pause the goal for Human QA after PR creation.

Blocked by

#632 — authorized received-file access and native file-operation integration.

Scope boundaries

No managed-original relocation, external-platform replacement, collaborative editor, version history or provider expansion. Reuse canonical file identity/current bytes and existing permission owners.

Activity

  1. added this to the v1.0 milestone on Oct 1, 2026
  2. added
    ready-for-agentFully specified, ready for an AFK agent
    area:fullstackEnd-to-end Host and Client implementation
    feature:workspaceWorkspace Grant and working-directory authority
    feature:messagingMessaging module and external providers
    scope:tracerHuman-testable implementation slice
    on Oct 1, 2026
  3. DoodleBears commented on Oct 1, 2026

    @DoodleBears
    ContributorAuthor

    Agent work claim

    • Agent: Codex
    • Task: codex/local/01a0f268-9db7-7702-b051-bc1bb94b22b9 (not a DSH Session)
    • Issue scope: trusted single-original access and explicitly requested native editing, current downloads/shared-reference proof, focused permission regressions and real-model E2E evidence; default independent save-as remains intact.
    • Branch: codex/633-original-attachment-edit
    • Started: 2026-10-01 07:22 UTC
    • State: active; Tracer: Process a received CSV ZIP in an authorized directory and return a new attachment #632 dependency is merged and closed. PR will pause for Human QA.
  4. added
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    in-progressHeld by an agent session; see Claim comment for who
    on Oct 1, 2026
  5. moved this from Todo to In Progress in Roadmapon Oct 1, 2026
  6. DoodleBears commented on Oct 1, 2026

    @DoodleBears
    ContributorAuthor

    Agent work outcome

    • Task: codex/local/01a0f268-9db7-7702-b051-bc1bb94b22b9 (not a DSH Session)
    • State: paused for Human QA
    • PR: feat: edit selected original attachments with native tools #653; commit 195d67f9359c4686e4d9c1e474020f61c43ac435 integrates main 7fe56622 without changing task scope.
    • Completed scope: exact-source original acquisition, existing approval for explicit editing, per-call source and native-path rechecks, turn cleanup, current shared-reference downloads and independent uploads.
    • Real-model proof: six successful native/application calls, zero failures and completed turn; actual browser download matches approved original bytes. Original/shared references remain approved after restart; independent identical upload and uploader source remain pending.
    • Evidence: three GitHub-rendered screenshots in the PR Change outline, verified loaded in a public browser.
    • Remaining scope: Human QA, exact-head CI/review, then separately authorized merge; external bridge slices remain under Hub: PersonaBot attachment file operations #631.

    Integrated full suite: 1671 passed / 1 existing skipped; typecheck, lint, ledgers, format and bundle build passed. Final-build cold restart and authenticated current-byte checks passed.

    Exact-head CI is running at verify run 36834781451; PR is currently mergeable. CI/review are not asserted complete.

  7. removed
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    on Oct 1, 2026
  8. DoodleBears commented on Oct 1, 2026

    @DoodleBears
    ContributorAuthor

    Agent work claim — Human-approved merge preparation

  9. added
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    on Oct 1, 2026
  10. DoodleBears commented on Oct 1, 2026

    @DoodleBears
    ContributorAuthor

    Agent work outcome — Human-approved merge completed

    Real-model native edit, actual browser download, current shared identity, independent upload/source preservation and Host restart proof remain in the PR with three publicly rendered screenshots. The newly merged Group departure behavior was retained in the prompt. No provider attachment support is claimed from this local proof.

    The two approved local slices under #631 are complete. Next qualification is a narrow Lark attachment acquire/process/same-source file-reply tracer, respecting #12's separately active claim and public provider contract boundary. Hub/provider issues remain open.

  11. removed
    agent:activeOne or more coding-agent tasks currently claim this issue; see claim comments for task IDs
    in-progressHeld by an agent session; see Claim comment for who
    on Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:fullstackEnd-to-end Host and Client implementationfeature:messagingMessaging module and external providersfeature:workspaceWorkspace Grant and working-directory authorityready-for-agentFully specified, ready for an AFK agentscope:tracerHuman-testable implementation slice

    Type

    No type

    Projects

    • Status
      Done

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions