[jgregoriorincon]

Plugin: IDE Barranquilla 0.1.1

What it does: Browses the catalogue of the Alcaldía de Barranquilla's geoviewer (GeoBAQ) and adds its services to the map: ArcGIS layers as features (read in pages, as protocol buffers when the layer offers them) or as a map image. Adding a layer takes the map to its extent. It has a panel with a searchable catalogue tree, and an IDE Barranquilla ribbon tab with reference layers (neighbourhoods, socioeconomic strata, localities, urban areas, facilities, Transmetro) and the Alcaldía's three vector basemaps.

• Entry: the ide-barranquilla.json from the ide-barranquilla-v0.1.1 release (package SHA-256 7f30e8b3…2883), with the categories Basemaps (the Alcaldía's basemaps), Cartography (the city's reference cartography: neighbourhoods, localities, strata), Data (the geoviewer's catalogue) and Vector (layers added as features).
• Permissions: edit (it adds layers and basemaps) and network. The catalogue is embedded in the plugin as an index, rebuilt weekly by the plugin's CI; at run time it reads only the services the user picks, all on the Alcaldía's own servers: miciudad.barranquilla.gov.co and its ArcGIS Online organisation (services3.arcgis.com/oGYAc07w6wsvgUYr). The national services the geoviewer cuts down to Barranquilla, and the maps it hides, are left out.
• minRustgisVersion 0.6.0.
• The Alcaldía's servers sit behind Cloudflare, which lets through clients whose User-Agent starts with RustGIS/ (an exception the Alcaldía made for RustGIS): the desktop app and rustgis-cli load the services; the web app, which can't set one, may not.
• Adding a layer again replaces only the layers the plugin added, known by a parameter of its own in their features query (rustgis-plugin=ide-barranquilla, which ArcGIS servers ignore), as IDE Bogotá does through the shared rustgis-ide-core.
• The plugin and its README say plainly that it is unofficial, and describe what it holds and how to use it (0.1.1).

How I tested it: RustGIS 0.6.0 desktop (Linux): with the map on the whole world, adding a layer takes the map to it. The CI and the release workflow built the package from the ide-barranquilla-v0.1.1 tag and ran the plugin's commands in rustgis-cli 0.6.0: the ten reference layers, downloaded from the Alcaldía's servers, and the three basemaps.

• The entry's source.url is a release asset of the plugin's repository, and sha256 is that file's.
• The plugin asks only for the permissions it uses.

Summary by CodeRabbit

• New Features
  • Added the Barranquilla IDE plugin to the registry. Its listing is categorized under Basemaps, Cartography, Data, and Vector, making it available alongside other registry entries in those categories. The listing also includes release information for the plugin.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The registry adds an ide-barranquilla entry with a RustGIS plugin release URL, SHA-256 checksum, and Basemaps, Cartography, Data, and Vector categories.

Changes: Plugin registry

Layer / File(s)	Summary
Add plugin registry record registry/ide-barranquilla.json	Adds the release URL, SHA-256 checksum, and Basemaps, Cartography, Data, and Vector categories.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Feature

Possibly related PRs

• opengeos/rustgis-plugins#56: Both PRs use source markers to replace catalogue-plugin layers without removing user-added layers for the same service.

Merge Risk: Minimal · up to aaa0a

No merge-blocking issue is established for the registry entry.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly identifies the main change: adding IDE Barranquilla to the registry.
Description check		The description includes the plugin name and repository, explains what the plugin does, reports how it was tested, and marks both required registry and permissions checks as complete.
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the registry row,
A plugin link is set to go.
Its checksum guards the release path,
Four categories fill its map.
Basemaps, data, vectors shine,
Cartography joins the line.}

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: eaa2f4ff-c6ca-4484-99cc-6a133408fd6f

Commits

Reviewing files that changed from the base of the PR and between e919b57 and aaa0a62.

Files selected for processing: 1

• registry/ide-barranquilla.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

[github-actions]

Code review

Bugs

• None found. The entry has id, source.url, source.sha256 and categories, and the id matches the package's plugin.json (ide-barranquilla, 0.1.0). I did not compare the SHA-256 myself. The trusted step reported that it verified it. (high)

Security

• The module's strings list layer.remove, layer.rename and layer.copyPath. The PR description explains layer.remove: it replaces only layers the plugin added, which it identifies by the rustgis-plugin=ide-barranquilla query parameter. I could not read the compiled code to confirm that the removal is limited this way. A maintainer may want to check it. (low-medium)
• network is a permission the plugin's code does not enforce, so the sandbox only gates host requests. The description says it reads only the Alcaldía's services. The module strings also name https://carreradelasrosas.com/, which is not an Alcaldía host, and 363 more URLs that module.txt does not show. The visible ones are miciudad.barranquilla.gov.co, plus the author's GitHub homepage. Please ask the author what carreradelasrosas.com is, presumably a catalogue entry, and confirm that every embedded URL belongs to the catalogue. (medium)
• No files permission is requested. edit matches what the plugin does, which is adding layers and basemaps. (high)

Performance

• Not assessable, because the module is compiled. The description says layers are read in pages. Whether huge layers fail with a clear message is untested. (low)

Quality

• The entry has the right shape. The source repository is linked as homepage and has a tagged release. (high)
• The web app may not load these services, because it can't set the RustGIS/ User-Agent. The PR description already says so. (info)

CLAUDE.md

• No guidelines apply to this change. The PR adds a registry entry only, with no plugin source, so it is in scope.

Plugin packages

id	version	runtime	package	module	permissions	module imports	commands named in strings
ide-barranquilla	0.1.0	wasm	129,888 B	502,691 B	edit, network	rustgis.call, rustgis.result	host.layers, host.notify, host.view, layer.addUrl, layer.copyPath, layer.remove, layer.rename, map.addTiles, map.zoomToExtent

Assessment of ide-barranquilla:

• The package was inspected. The imports are allowed, and the exports are memory, rustgis_alloc and rustgis_call.
• The manifest is consistent with the entry. The description matches the commands, which are catalogue browsing, adding features or a map image, and reference-layer shortcuts.
• The commands named in strings fit the stated purpose. These are layer.addUrl and map.addTiles to add layers, map.zoomToExtent and host.view to move the view, and layer.remove to replace its own layers.
• Source is reachable at the homepage (github.com/jgregoriorincon/rustgis-ide-barranquilla).
• The one open point is the non-Alcaldía URL carreradelasrosas.com, described under Security. Everything else is as expected.

Change size

Compared: the whole PR against main (merge base f7bad81 to PR head e919b57).

Metric	Value
Files changed	1 (1 added, 0 modified, 0 removed, 0 renamed)
Size before	0 B
Size after	362 B
Total size change	+362 B

File	Before	After	Change
registry/ide-barranquilla.json	0 B	362 B	+362 B

[github-actions]

🧩 Plugin preview

Removed: the pull request is closed.

[jgregoriorincon]

On the embedded URLs: they come from the catalogue index the plugin carries (ide-barranquilla/index.json, 230 services). Its service addresses are all on the Alcaldía's servers: 518 URLs on miciudad.barranquilla.gov.co (services and their WMS/WFS/WMTS alternates) and 4 on its ArcGIS Online organisation (services3.arcgis.com/oGYAc07w6wsvgUYr). https://carreradelasrosas.com/ is the one other host, and it is text, not a service: the description of the catalogue's "Carrera de las Rosas" map (id 9-4, itself on miciudad.barranquilla.gov.co/gis/rest/services/recreacionydeporte/carreradelasrosas/MapServer) says to see that site for tickets. The plugin shows it on the service's card and never fetches it.

On layer.remove: the plugin removes a layer only when its source, from layer.copyPath, is that service's features query with rustgis-plugin=ide-barranquilla (ide_core::layers::added_by, tested in rustgis-ide-core); the plugin's source is in rustgis-ide-barranquilla.

[jgregoriorincon]

@coderabbitai review

[coderabbitai]

Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[github-actions]

Code review

Bugs

• None found in the diff. It adds one file, registry/ide-barranquilla.json, with an id, a source (URL and SHA-256) and categories. The id matches the package manifest (ide-barranquilla, 0.1.1). Confidence: high.

Security

• No plugin source is added. Only a registry entry pointing at a release asset of the plugin's own repository, as the scope rules ask. Confidence: high.
• The network permission matches the stated purpose (reading ArcGIS services). edit matches adding layers and basemaps. There is no files permission. Confidence: medium-high.
• layer.remove, layer.rename and layer.copyPath appear in the module's strings. The PR says "adding a layer again replaces only the layers the plugin added", which explains layer.remove. I couldn't confirm from the compiled module that it only removes its own layers. A maintainer may want to check this, because removing user layers would be destructive. Confidence: low-medium.
• Every host named in the module strings is miciudad.barranquilla.gov.co. Two other strings appear: https://carreradelasrosas.com/ (presumably a catalogue entry) and a malformed https://http://… fragment that looks like concatenated string constants, not a real URL. The PR says services3.arcgis.com is also used, but that host isn't in the strings, so it is probably built at run time. Confidence: medium.

Performance

• The module is 502 KB and the catalogue is embedded, so loading it is somewhat heavy. Features are read in pages. I could not check how it behaves on a huge layer. Confidence: low.

Quality

• The PR says the Alcaldía's Cloudflare allows only clients whose User-Agent starts with RustGIS/, so the web app may fail to load these services. That is documented in the PR. Confirm the plugin README says so too. Confidence: medium.
• Only the entry's source is verified here, not the homepage or repository. The manifest homepage is https://github.com/jgregoriorincon/rustgis-ide-barranquilla, which matches the release URL. Confidence: high.

CLAUDE.md

• No CLAUDE.md guidelines apply to a registry entry.

Plugin packages

id	version	runtime	package	module	permissions	module imports	commands named in strings
ide-barranquilla	0.1.1	wasm	130,016 B	502,867 B	edit, network	rustgis.call, rustgis.result	host.layers, host.notify, host.view, layer.addUrl, layer.copyPath, layer.remove, layer.rename, map.addTiles, map.zoomToExtent

Assessment: The package was inspected and its SHA-256 verified. The exports memory, rustgis_alloc and rustgis_call are present, and the imports are allowed ones. The manifest's name, version and description match the entry, and the description says it is unofficial. The commands fit the stated purpose: add URL layers, add tiles, zoom to an extent, and replace the plugin's own layers. The layer.remove/layer.rename/layer.copyPath caveat above is the one item to confirm. The module is compiled and can't be read here, so the source repository linked from homepage is where its behaviour can be checked.

Change size

Compared: the whole PR against main (merge base f7bad81 to PR head aaa0a62).

Metric	Value
Files changed	1 (1 added, 0 modified, 0 removed, 0 renamed)
Size before	0 B
Size after	362 B
Total size change	+362 B

File	Before	After	Change
registry/ide-barranquilla.json	0 B	362 B	+362 B