[docs] Port changelog for release 1.2.1#253
Conversation
📝 WalkthroughWalkthroughThe pull request adds a changelog entry for version 1.2.1 dated 2026-04-09 to the CHANGELOG.rst file. The entry documents a bugfix under a Bugfixes subsection that addresses UCI name escaping in the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CHANGELOG.rst`:
- Line 10: Update the changelog entry text to reference the actual script name
used in the codebase: replace "uci-autoname script" with "openwisp-uci-autoname
script" so the line reads "Fixed UCI name escaping in openwisp-uci-autoname
script"; locate the string in CHANGELOG.rst and edit that entry to match the
filename and installation references
(openwisp-config/files/sbin/openwisp-uci-autoname and
$(1)/usr/sbin/openwisp-uci-autoname) and keep formatting consistent with other
entries.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 70f75e31-e904-4df2-ba8a-705469529c89
📒 Files selected for processing (1)
CHANGELOG.rst
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Analyze (actions)
- GitHub Check: Kilo Code Review
| Bugfixes | ||
| ~~~~~~~~ | ||
|
|
||
| - Fixed UCI name escaping in uci-autoname script |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Find the actual script name in the codebase to verify changelog accuracy
# Search for script files with 'autoname' in the name
echo "=== Searching for autoname scripts ==="
fd -t f -e sh 'autoname'
# Search for references to both script names in the codebase
echo -e "\n=== Searching for 'openwisp-uci-autoname' references ==="
rg -l 'openwisp-uci-autoname' --type sh
echo -e "\n=== Searching for 'uci-autoname' references (excluding openwisp- prefix) ==="
rg -l '\buci-autoname\b' --type sh -g '!CHANGELOG.rst'Repository: openwisp/openwisp-config
Length of output: 242
🏁 Script executed:
#!/bin/bash
# Broader search to find script references
echo "=== Checking for scripts in common locations ==="
find . -type f \( -name "*autoname*" -o -name "bin" -o -name "scripts" \) 2>/dev/null | head -20
echo -e "\n=== Searching all file types for 'openwisp-uci-autoname' ==="
rg 'openwisp-uci-autoname' --type-list | head -5
rg 'openwisp-uci-autoname' -g '!.git' 2>/dev/null | head -10
echo -e "\n=== Searching all file types for 'uci-autoname' (broader match) ==="
rg 'uci-autoname' -g '!.git' -g '!CHANGELOG.rst' 2>/dev/null | head -10
echo -e "\n=== Checking setup.py or package metadata ==="
fd -t f '(setup|pyproject|setup\.cfg)' | xargs cat 2>/dev/null | grep -A 5 -B 5 'autoname' 2>/dev/null
echo -e "\n=== Checking directory structure ==="
ls -la | grep -E '^d'
find . -maxdepth 2 -type d -name '*bin*' -o -name '*script*' 2>/dev/nullRepository: openwisp/openwisp-config
Length of output: 2061
Fix the script name to match codebase references.
The changelog entry refers to "uci-autoname script", but the actual script in the codebase is named "openwisp-uci-autoname" (with the openwisp- prefix). This is evident from the script location (openwisp-config/files/sbin/openwisp-uci-autoname.lua), Makefile installation ($(1)/usr/sbin/openwisp-uci-autoname), and previous changelog entries (lines 177, 190) which all reference "openwisp-uci-autoname". Update the entry to: "Fixed UCI name escaping in openwisp-uci-autoname script"
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@CHANGELOG.rst` at line 10, Update the changelog entry text to reference the
actual script name used in the codebase: replace "uci-autoname script" with
"openwisp-uci-autoname script" so the line reads "Fixed UCI name escaping in
openwisp-uci-autoname script"; locate the string in CHANGELOG.rst and edit that
entry to match the filename and installation references
(openwisp-config/files/sbin/openwisp-uci-autoname and
$(1)/usr/sbin/openwisp-uci-autoname) and keep formatting consistent with other
entries.
Note: This PR was automatically generated by the
openwisp-utilsreleaser tool.