feat!: make chandler composer dependency - #1718
Merged
Merged
Conversation
WerySkok
force-pushed
the
restructurisation
branch
5 times, most recently
from
August 3, 2026 11:46
5fa462c to
9ca3a8d
Compare
WerySkok
force-pushed
the
restructurisation
branch
from
August 3, 2026 15:18
f01853f to
5460fda
Compare
veselcraft
approved these changes
Aug 7, 2026
veselcraft
left a comment
Member
There was a problem hiding this comment.
да вроде адекватно выглядит
WerySkok
added a commit
to OpenVK/chandler
that referenced
this pull request
Aug 8, 2026
> НАСТОЯЩИЙ МАТЕРИАЛ (ИСХОДНЫЙ КОД) СОЗДАН И (ИЛИ) ОТРЕДАКТИРОВАН ПРИ УЧАСТИИ ИИ-АГЕНТА И (ИЛИ) КАСАЕТСЯ ДЕЯТЕЛЬНОСТИ ТАКОГО ИИ-АГЕНТА Resolves #17. Мержить синхронно с аналогичным PR в OpenVK. Я нагенерил. Особенности: - Extensions В С Ё - CommitCaptcha перенесена внутрь (с заменой шрифта с San Francisco на Inter) - Есть пример приложения в новом формате Скрипт миграции для OpenVK был создан в OpenVK/openvk#1718
- Replace 500-line auto-bootstrap with explicit bootstrap_openvk() function - Move helper functions (tr, ovk_proc_strtr, etc.) and boot closure to ovk-init.php (loaded by ExtensionManager as init script) - Remove manifest.yml (no longer needed) - Update chandler_loader.php to use new bootstrap_openvk()
- Replace deprecated property with PHP 8 #[AsCommand] attribute - Fix UpgradeCommand to resolve Chandler path from vendor directory instead of old extension directory structure
Switch from composite-repo layout to Chandler as a standard Composer path dependency.
- Remove separate chandler.yml — settings merged into openvk.yml (chandler: section) - Remove commitcaptcha references and chandler.example.yml - Update all paths from /opt/chandler/extensions/available/openvk to /opt/openvk - Update Docker build for Chandler as build context (not git clone) - Update CI workflow for new Chandler structure - Update install instructions in README
- Add Chandler checkout step to all CI workflows
(analyse.yaml, build.yaml, test-screenshots.yaml)
- Replace hardcoded /opt/chandler in build-context with
${{ github.workspace }}/chandler from checkout
- Add build-contexts input to docker/build-push-action in build.yaml
- Make docker-compose.test.yml and docker-compose.dev.yml use
CHANDLER_PATH env var with /opt/chandler as fallback
- Remove dead mkdir for extensions/available, extensions/enabled
from openvk.Dockerfile
OpenVK now lives at /opt/openvk, not inside the Chandler extensions directory. Add its own document root entry point.
- bin/upgrade-structure.php handles old→new structure migration - Supports --extract (move OVK out of extensions/) and in-place modes - Pre-flight checks (perms, disk space), backup with rsync, config merge, composer install, validation - --dry-run and --force flags for safe operation - Moved from Chandler repo (replaced with forwarding stub)
Use the non-resolved path for structure detection (inside extensions/) and the real path for backup, config merge, cleanup, and validation.
- Allow --target-ovk to be a symlink pointing to the current OVK source - Load all available autoloaders until Symfony YAML becomes available
This makes this class visible to PHPStan
I swear I grepped these filenames and they were not used anywhere, even FreeBSD install script
WerySkok
force-pushed
the
restructurisation
branch
from
August 8, 2026 13:49
5460fda to
1d2a8d8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Мержить после аналогичного PR в Chandler и его публикации в Packagist.
Меняет структуру установки местами. Раньше нужно было ставить Chandler, а внутри него OpenVK как расширение. Теперь OpenVK становится главным — теперь достаточно установить OpenVK, а Chandler, как и остальные зависимости, теперь ставится через Composer.
Я нагенерил. Вроде локальный инстанс работает. Решение назревало давно, но есть один слабый момент — миграция. Нужно будет вытащить папку OpenVK наружу и применить все изменения. Вроде нейронка нагенерила мне скрипт для миграции, вроде работает.
Если будете тестить в виртуалке — делайте снапшот, чтобы можно было откатиться)
Особенности:
Планы: