<?xml version="1.0" encoding="UTF-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" xml:lang="en">
    <title>Colas.Nahaboo.net - github</title>
    <subtitle>Colas Nahaboo personal site, with discussions about programming code, web and computing topics, surfing and SUPing, and various musings.</subtitle>
    <link rel="self" type="application/atom+xml" href="https://colas.nahaboo.net/tags/github/atom.xml"/>
    <link rel="alternate" type="text/html" href="https://colas.nahaboo.net"/>
    <generator uri="https://www.getzola.org/">Zola</generator>
    <updated>2026-08-25T00:00:00+00:00</updated>
    <id>https://colas.nahaboo.net/tags/github/atom.xml</id>
    <entry xml:lang="en">
        <title>My first AI-assisted pull request accepted</title>
        <published>2026-08-25T00:00:00+00:00</published>
        <updated>2026-08-25T00:00:00+00:00</updated>
        
        <author>
          <name>Unknown</name>
        </author>
        
        <link rel="alternate" type="text/html" href="https://colas.nahaboo.net/code/first-ai-assisted-accepted-pr/"/>
        <id>https://colas.nahaboo.net/code/first-ai-assisted-accepted-pr/</id>
        
        <content type="html" xml:base="https://colas.nahaboo.net/code/first-ai-assisted-accepted-pr/">&lt;p&gt;&lt;img src=&#39;https://nicefox.net/logo/highres%20fox.png&#39; width=120px align=right&gt;I just got my first AI-assisted pull request accepted and merged. I must say, I am quite pleased — and a bit humbled: the AI wrote the code, I did the dull part: project manager.&lt;/p&gt;
&lt;p&gt;The project is &lt;a rel=&quot;noopener external&quot; target=&quot;_blank&quot; href=&quot;https://github.com/co-l/openfox&quot;&gt;Openfox&lt;/a&gt;, a smart local-LLM-first agentic coding assistant. I found a bug: it did not see projects that are symlinks to directories. I keep my git repos in various places, and symlink them into the folder Openfox scans — but &lt;code&gt;Dirent.isDirectory()&lt;/code&gt; just tested if a file was a directory, so it failed to detect that a symlink could be a link to an actual directory. Thus symlinked project folders were invisible in the directory browser, the workspace listing, the orphaned-workspace scan, and the plugin listing.&lt;/p&gt;
&lt;p&gt;Following my own &lt;a href=&quot;/blog/dont-do-pull-request-do-prompt-suggest/&quot;&gt;prompt request idea&lt;/a&gt;, I asked Openfox itself — running DeepSeek V4 Flash — to analyze and fix the bug on a local copy. It found the root cause, proposed a shared &lt;code&gt;isDirectoryEntry&lt;/code&gt; helper that follows symlinks via &lt;code&gt;stat()&lt;/code&gt; (excluding broken and cyclic links), and even produced a careful analysis of Windows junction points. I opened the &lt;a rel=&quot;noopener external&quot; target=&quot;_blank&quot; href=&quot;https://github.com/co-l/openfox/issues/243&quot;&gt;issue&lt;/a&gt; and offered the prompt to the maintainer.&lt;/p&gt;
&lt;p&gt;Then I decided on my own to submit a real pull request anyway. My prompt-request post concedes there are cases where a PR is still better — this was one: the fix was small and well-contained, tests existed, and I had a full suite to prove it. Leaving all that on the maintainer would have been rude.&lt;/p&gt;
&lt;p&gt;So &lt;a rel=&quot;noopener external&quot; target=&quot;_blank&quot; href=&quot;https://github.com/co-l/openfox/pull/245&quot;&gt;PR #245&lt;/a&gt; it was: the &lt;code&gt;isDirectoryEntry&lt;/code&gt; helper used at all four call sites, with 43 new tests covering symlinked workspaces and the directories API — broken symlinks excluded, and a Windows skip where needed. The full unit suite passed (over 4000 tests), static checks were clean, and I rebased onto &lt;code&gt;develop&lt;/code&gt; three times as the upstream moved. Merged as &lt;code&gt;37e78208&lt;/code&gt;, ships in v2.0.125.&lt;/p&gt;
&lt;p&gt;Which prompted me to think: the AI used an AI tool - openfox — co-made by AI to generate the code, I did the specs, the validation of the diagnostics, the tests, the rebases, the cross-platform care — and the judgment call to ship a proper PR with full coverage instead of prompt slop. I acted as an auditor, the role that &lt;a rel=&quot;noopener external&quot; target=&quot;_blank&quot; href=&quot;https://www.cringely.com/2026/07/16/the-auditors-opinion/&quot;&gt;Cringely is saying is our added value for us humans&lt;/a&gt;.&lt;/p&gt;
</content>
        
    </entry>
</feed>
