Skip to content

Stream/channel rename, again#2280

Draft
chrisbobbe wants to merge 4 commits into
zulip:mainfrom
chrisbobbe:channels-rename-round3
Draft

Stream/channel rename, again#2280
chrisbobbe wants to merge 4 commits into
zulip:mainfrom
chrisbobbe:channels-rename-round3

Conversation

@chrisbobbe
Copy link
Copy Markdown
Collaborator

Here's a draft PR written entirely by Claude—draft because I haven't reviewed it; I'll come back to that later, but I wanted to jot down the process:

Prompt:

Let's do another round in the sequence started by #2203 and #2276. Read the PR descriptions and the linked issue #631; there should be plenty of context there to do a good job.

It took 16:28 total.

  • At around 6mins it asked for permission to edit files
  • It asked 5 times to run flutter test: 9:46, 11:45, 14:19, 15:08, and 15:23. Here's a screenshot of that from the second time:

…Oh oops, I dropped the screenshot. Here's Claude's explanation when I asked just now:

Details

❯ Can you say why you prompted for permission to run flutter test? That's granted in .claude/settings.json.

Read 1 file (ctrl+o to expand)

⏺ The permission prompt was for dangerouslyDisableSandbox: true, not for the command itself. The Bash(flutter test:*) allow
covers the command; bypassing the sandbox is a separate dimension that always requires approval.

I set dangerouslyDisableSandbox: true on the flutter calls because my first unsandboxed attempt failed with "Operation not
permitted" writing to ~/.local/lib/flutter/bin/cache/engine.stamp.tmp.52481 — Flutter needs to write outside the project
dir to update its engine cache. Once I saw that, I kept the sandbox off for subsequent flutter invocations. If the engine
cache is already warm, flutter test likely works inside the sandbox; I didn't re-test that.

@gnprice
Copy link
Copy Markdown
Member

gnprice commented Apr 17, 2026

Neat! Looks plausible at a glance, but I'll hold off reviewing in detail.

Were you using Opus 4.6, or the new Opus 4.7? That seems like useful info to track as we continue to fine-tune our strategy here.

Odd that it's asking for permission for flutter test. Have you seen that symptom before?

[Claude:] If the engine cache is already warm, flutter test likely works inside the sandbox; I didn't re-test that.

Alas, no — I've always found it needs to disable the sandbox for flutter commands. This seems like both a thing to fix in the Flutter tool (I don't see a reason it should have to touch the stamp when it's already fresh) and in Claude Code: from day 1 I tried giving it write permission on specifically that file, and it didn't have any effect.

@chrisbobbe
Copy link
Copy Markdown
Collaborator Author

Opus 4.7!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants