While I like the git and CI setup, I'm wondering where AI helps?
The Don't do git push in Claude.md is stopped by the hook. So why keep it in Claude.md?
/pr could be a make command or whatever script runner you prefer. You can integrate an LLM call in the command to generate the message.
From what I read between the lines is that you are pushing/leading people to the command line/terminal. If people want to use Claude why not let them use the desktop app?
With the desktop app one of the session setup parts is the directory. So no need for cd.
And then the Don't do git push line makes sense, otherwise the Claude app is burning tokens getting the same git hook message every time.
Give visual people a GUI. We are used to seeing text all the time, but it is because we are weird (in a good way).
I host the Arrested DevOps podcast. Ex-DevOpsDays organizer. Head of DevRel at Tiger Data: Postgres, time-series. Formerly pretending to be a Go dev. Currently pretending to understand robotics.
I think there's something in between the lines getting lost here, I'm not saying that the AI being there is helping the code that Tanya writes herself be checked, this is to make sure that the process that the AI uses that she (and myself, and others) is asking it to build with is following the rules.
I would never sit there and say you should layer an AI on to make sure that code that you are writing yourself is deployed correctly. We were doing that before there was AI. This is how we have the agents themselves follow rules. Yes many of those rules don't require something built into the agent but the PR flow is actually not as simple as something an make command could do. It may be becoming a little oversimplified in my ability to write this into a blog post but it includes loops. It includes certain decision trees. It's not just simply a bash script... nothing wrong with bash scripts by the way.
Also there's a little bit of a joke in here about the terminal but I am not the one that would ever tell anybody you should just do everything at the command line. This works exactly the same way if you're using Claude code on the desktop. Also even if you're using Claude code on the desktop there's at least one part where you have to use the terminal, which is to get the repo on your machine. There is no way in Claude code on the desktop, as an example, to say go get this from this GitHub repo on the web. Although if you are using Claude code in the web UI, it does have that, which has a nice advantage but that's a different story.
Anyway yes I know how to build all of those things without using AI and this was not about saying how to use AI to do the things we've already been doing with these other tools. This is saying if you are using AI as your coding agent, here's how you can embed some of these things.
I host the Arrested DevOps podcast. Ex-DevOpsDays organizer. Head of DevRel at Tiger Data: Postgres, time-series. Formerly pretending to be a Go dev. Currently pretending to understand robotics.
Also if it wasn't clear in my comment that I first posted, I appreciate you taking the time to read this and comment on it and help me see things that I wasn't framing clearly in my post! I appreciate it
I think the emphasis on Claude.md and skills feels to me like you are downplaying your work.
It takes up a large part of the post, while it is only one of the situations that benefit from the great setup.
When you are opening with "person that doesn't know cd" and mentioning Claude code and Claude.md a few times, I'm thinking terminal, so that can be chalked up to my perceptive.
not as simple as something an make command could do
Make can run anything, it is not only bash scripts.
The thing I wanted to make clear was that the PR procedure doesn't need to burn tokens. Use AI where it makes the process easier.
I'm commenting because your post has valuable information. It just felt a bit off for me, and I wanted to address that.
I host the Arrested DevOps podcast. Ex-DevOpsDays organizer. Head of DevRel at Tiger Data: Postgres, time-series. Formerly pretending to be a Go dev. Currently pretending to understand robotics.
I love it! Also after I read your comment it made me go back through our PR skill and look for ways to make it more token efficient so thanks for that! You are saving my usage rates !
Like everything else in this business of ours, there is no one panacea for everything and it isn't all solved with one tool but you know that as well as I do :)
I host the Arrested DevOps podcast. Ex-DevOpsDays organizer. Head of DevRel at Tiger Data: Postgres, time-series. Formerly pretending to be a Go dev. Currently pretending to understand robotics.
Also to be fair the whole thing about not knowing the cd command is a little bit click-baity and a little bit for fun, mostly because of that exact conversation we had the morning that I was starting to write this post. Tanya and I both thought it was hilarious and I joked that that is what I was going to lead with. She said, "No you absolutely should do that. I love it so much."
I host the Arrested DevOps podcast. Ex-DevOpsDays organizer. Head of DevRel at Tiger Data: Postgres, time-series. Formerly pretending to be a Go dev. Currently pretending to understand robotics.
The Don't do git push in Claude.md is stopped by the hook. So why keep it in Claude.md?
Yeah I think I didn’t write that part of the post very clearly; that’s exactly correct and it shouldn’t be there. It started there for me/us and then it became apparent that didn’t work.
And the CLAUDE.md doesn’t say “don’t push to main” it says “don’t push to origin directly/try to make up your own way of doing PRs, use the skill”. And guess what, it still sometimes doesn’t use the skill. But it’s not the end of the world if it doesn’t use the skill; it just isn’t as good. Many of the things the skill does (from a safety perspective) will be done by the CI as well; but having the agent do them before the push saves having the CI then fail it, and wasting the time.
The critical safety thing that cannot be done differently is the pushing to the wrong branch, that's why that one is a hook and I think what I was trying to illustrate by the golden rules was almost a little bit of irony that it's called a golden rule and that's almost the most direct way to refer to them in something like a Claude.md. Even then they can get bypassed so that's why things are layered.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
While I like the git and CI setup, I'm wondering where AI helps?
The
Don't do git pushin Claude.md is stopped by the hook. So why keep it in Claude.md?/prcould be a make command or whatever script runner you prefer. You can integrate an LLM call in the command to generate the message.From what I read between the lines is that you are pushing/leading people to the command line/terminal. If people want to use Claude why not let them use the desktop app?
With the desktop app one of the session setup parts is the directory. So no need for
cd.And then the
Don't do git pushline makes sense, otherwise the Claude app is burning tokens getting the same git hook message every time.Give visual people a GUI. We are used to seeing text all the time, but it is because we are weird (in a good way).
I think there's something in between the lines getting lost here, I'm not saying that the AI being there is helping the code that Tanya writes herself be checked, this is to make sure that the process that the AI uses that she (and myself, and others) is asking it to build with is following the rules.
I would never sit there and say you should layer an AI on to make sure that code that you are writing yourself is deployed correctly. We were doing that before there was AI. This is how we have the agents themselves follow rules. Yes many of those rules don't require something built into the agent but the PR flow is actually not as simple as something an
makecommand could do. It may be becoming a little oversimplified in my ability to write this into a blog post but it includes loops. It includes certain decision trees. It's not just simply a bash script... nothing wrong with bash scripts by the way.Also there's a little bit of a joke in here about the terminal but I am not the one that would ever tell anybody you should just do everything at the command line. This works exactly the same way if you're using Claude code on the desktop. Also even if you're using Claude code on the desktop there's at least one part where you have to use the terminal, which is to get the repo on your machine. There is no way in Claude code on the desktop, as an example, to say go get this from this GitHub repo on the web. Although if you are using Claude code in the web UI, it does have that, which has a nice advantage but that's a different story.
Anyway yes I know how to build all of those things without using AI and this was not about saying how to use AI to do the things we've already been doing with these other tools. This is saying if you are using AI as your coding agent, here's how you can embed some of these things.
Also if it wasn't clear in my comment that I first posted, I appreciate you taking the time to read this and comment on it and help me see things that I wasn't framing clearly in my post! I appreciate it
I think the emphasis on Claude.md and skills feels to me like you are downplaying your work.
It takes up a large part of the post, while it is only one of the situations that benefit from the great setup.
When you are opening with "person that doesn't know cd" and mentioning Claude code and Claude.md a few times, I'm thinking terminal, so that can be chalked up to my perceptive.
Make can run anything, it is not only bash scripts.
The thing I wanted to make clear was that the PR procedure doesn't need to burn tokens. Use AI where it makes the process easier.
I'm commenting because your post has valuable information. It just felt a bit off for me, and I wanted to address that.
I love it! Also after I read your comment it made me go back through our PR skill and look for ways to make it more token efficient so thanks for that! You are saving my usage rates !
Like everything else in this business of ours, there is no one panacea for everything and it isn't all solved with one tool but you know that as well as I do :)
Also to be fair the whole thing about not knowing the
cdcommand is a little bit click-baity and a little bit for fun, mostly because of that exact conversation we had the morning that I was starting to write this post. Tanya and I both thought it was hilarious and I joked that that is what I was going to lead with. She said, "No you absolutely should do that. I love it so much."Yeah I spotted that as soon I started reading. There is nothing wrong trying to attract readers.
Yeah I think I didn’t write that part of the post very clearly; that’s exactly correct and it shouldn’t be there. It started there for me/us and then it became apparent that didn’t work.
And the
CLAUDE.mddoesn’t say “don’t push to main” it says “don’t push to origin directly/try to make up your own way of doing PRs, use the skill”. And guess what, it still sometimes doesn’t use the skill. But it’s not the end of the world if it doesn’t use the skill; it just isn’t as good. Many of the things the skill does (from a safety perspective) will be done by the CI as well; but having the agent do them before the push saves having the CI then fail it, and wasting the time.The critical safety thing that cannot be done differently is the pushing to the wrong branch, that's why that one is a hook and I think what I was trying to illustrate by the golden rules was almost a little bit of irony that it's called a golden rule and that's almost the most direct way to refer to them in something like a Claude.md. Even then they can get bypassed so that's why things are layered.