@unito/integration-cli 0.64.3 → 0.64.5

package/dist/test/hooks/updateNotifier.test.js

@@ -0,0 +1,356 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const tslib_1 = require("tslib");
+const strict_1 = tslib_1.__importDefault(require("node:assert/strict"));
+const child_process_1 = tslib_1.__importDefault(require("child_process"));
+const fs_1 = tslib_1.__importDefault(require("fs"));
+const os_1 = tslib_1.__importDefault(require("os"));
+const path_1 = tslib_1.__importDefault(require("path"));
+const sinon_1 = tslib_1.__importDefault(require("sinon"));
+const updateNotifier_1 = require("../../src/hooks/init/updateNotifier");
+function makeFakeSpawn() {
+    const fakeChild = { unref: sinon_1.default.stub() };
+    return sinon_1.default.stub().returns(fakeChild);
+}
+function makeDeps(cacheFile, overrides) {
+    return {
+        cacheFile,
+        spawn: makeFakeSpawn(),
+        log: sinon_1.default.stub(),
+        upgrade: sinon_1.default.stub(),
+        ...overrides,
+    };
+}
+describe('updateNotifier', () => {
+    afterEach(() => {
+        sinon_1.default.restore();
+    });
+    describe('isNewerVersion', () => {
+        it('returns true when latest major is higher', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('2.0.0', '1.0.0'), true);
+        });
+        it('returns true when latest minor is higher', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('1.2.0', '1.1.0'), true);
+        });
+        it('returns true when latest patch is higher', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('1.0.2', '1.0.1'), true);
+        });
+        it('returns false when versions are equal', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('1.0.0', '1.0.0'), false);
+        });
+        it('returns false when current is newer', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('1.0.0', '2.0.0'), false);
+        });
+        it('returns false when current minor is newer', () => {
+            strict_1.default.equal((0, updateNotifier_1.isNewerVersion)('1.1.0', '1.2.0'), false);
+        });
+    });
+    describe('readCache', () => {
+        let tmpDir;
+        beforeEach(() => {
+            tmpDir = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'update-notifier-test-'));
+        });
+        afterEach(() => {
+            fs_1.default.rmSync(tmpDir, { recursive: true });
+        });
+        it('returns parsed cache data from file', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const cacheData = { latest: '1.2.3', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(cacheData));
+            const result = (0, updateNotifier_1.readCache)(cacheFile);
+            strict_1.default.deepEqual(result, cacheData);
+        });
+        it('returns null when cache file does not exist', () => {
+            const result = (0, updateNotifier_1.readCache)('/nonexistent/path/cache.json');
+            strict_1.default.equal(result, null);
+        });
+        it('returns null when cache file is invalid JSON', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            fs_1.default.writeFileSync(cacheFile, 'not valid json');
+            const result = (0, updateNotifier_1.readCache)(cacheFile);
+            strict_1.default.equal(result, null);
+        });
+        it('returns null when cache JSON has wrong shape', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify({ foo: 'bar' }));
+            const result = (0, updateNotifier_1.readCache)(cacheFile);
+            strict_1.default.equal(result, null);
+        });
+        it('returns null when latest is not a string', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify({ latest: 123, checkedAt: Date.now() }));
+            const result = (0, updateNotifier_1.readCache)(cacheFile);
+            strict_1.default.equal(result, null);
+        });
+    });
+    describe('isCacheStale', () => {
+        it('returns true when cache is null', () => {
+            strict_1.default.equal((0, updateNotifier_1.isCacheStale)(null), true);
+        });
+        it('returns true when cache is older than interval', () => {
+            const cache = { latest: '1.0.0', checkedAt: Date.now() - 2 * 60 * 60 * 1000 };
+            strict_1.default.equal((0, updateNotifier_1.isCacheStale)(cache, 60 * 60 * 1000), true);
+        });
+        it('returns false when cache is within interval', () => {
+            const cache = { latest: '1.0.0', checkedAt: Date.now() - 30 * 60 * 1000 };
+            strict_1.default.equal((0, updateNotifier_1.isCacheStale)(cache, 60 * 60 * 1000), false);
+        });
+    });
+    describe('formatNotification', () => {
+        it('contains version numbers', () => {
+            const plain = (0, updateNotifier_1.stripAnsi)((0, updateNotifier_1.formatNotification)('0.64.4', '0.65.0'));
+            (0, strict_1.default)(plain.includes('0.64.4'), 'should include current version');
+            (0, strict_1.default)(plain.includes('0.65.0'), 'should include latest version');
+        });
+        it('contains npm package link', () => {
+            const plain = (0, updateNotifier_1.stripAnsi)((0, updateNotifier_1.formatNotification)('0.64.4', '0.65.0'));
+            (0, strict_1.default)(plain.includes('https://www.npmjs.com/package/@unito/integration-cli'), 'should include npm URL');
+        });
+        it('contains upgrade command', () => {
+            const plain = (0, updateNotifier_1.stripAnsi)((0, updateNotifier_1.formatNotification)('0.64.4', '0.65.0'));
+            (0, strict_1.default)(plain.includes('integration-cli upgrade'), 'should include upgrade command');
+        });
+        it('renders a box border', () => {
+            const msg = (0, updateNotifier_1.formatNotification)('0.64.4', '0.65.0');
+            (0, strict_1.default)(msg.includes('\u2500'), 'should have horizontal border');
+            (0, strict_1.default)(msg.includes('\u250c'), 'should have top-left corner');
+            (0, strict_1.default)(msg.includes('\u2510'), 'should have top-right corner');
+            (0, strict_1.default)(msg.includes('\u2514'), 'should have bottom-left corner');
+            (0, strict_1.default)(msg.includes('\u2518'), 'should have bottom-right corner');
+        });
+    });
+    describe('spawnBackgroundCheck', () => {
+        it('spawns a detached child process', () => {
+            const spawn = makeFakeSpawn();
+            const deps = makeDeps('/tmp/cache.json', { spawn });
+            (0, updateNotifier_1.spawnBackgroundCheck)(deps);
+            (0, strict_1.default)(spawn.calledOnce, 'should spawn a process');
+            strict_1.default.equal(spawn.firstCall.args[0], process.execPath);
+            const options = spawn.firstCall.args[2];
+            strict_1.default.equal(options.detached, true);
+            strict_1.default.equal(options.stdio, 'ignore');
+        });
+        it('passes correct cache file to the spawned script', () => {
+            const spawn = makeFakeSpawn();
+            const deps = makeDeps('/custom/path/cache.json', { spawn });
+            (0, updateNotifier_1.spawnBackgroundCheck)(deps);
+            const script = spawn.firstCall.args[1][1];
+            (0, strict_1.default)(script.includes('/custom/path/cache.json'), 'script should use the provided cache file');
+            (0, strict_1.default)(script.includes('/custom/path'), 'script should use the provided cache dir');
+        });
+        it('unrefs the spawned process so it does not block exit', () => {
+            const fakeChild = { unref: sinon_1.default.stub() };
+            const spawn = sinon_1.default.stub().returns(fakeChild);
+            const deps = makeDeps('/tmp/cache.json', { spawn });
+            (0, updateNotifier_1.spawnBackgroundCheck)(deps);
+            (0, strict_1.default)(fakeChild.unref.calledOnce, 'should unref the child process');
+        });
+    });
+    describe('checkForUpdate', () => {
+        let tmpDir;
+        let savedCI;
+        let savedArgv;
+        beforeEach(() => {
+            tmpDir = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'update-notifier-test-'));
+            // Unset CI so checkForUpdate doesn't skip (test helper sets CI=1)
+            savedCI = process.env.CI;
+            delete process.env.CI;
+            savedArgv = process.argv;
+        });
+        afterEach(() => {
+            process.argv = savedArgv;
+            fs_1.default.rmSync(tmpDir, { recursive: true });
+            if (savedCI !== undefined) {
+                process.env.CI = savedCI;
+            }
+            else {
+                delete process.env.CI;
+            }
+        });
+        it('calls upgrade when newer version is cached', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '99.0.0', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const upgrade = sinon_1.default.stub();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn: makeFakeSpawn(), log, upgrade });
+            (0, strict_1.default)(upgrade.calledOnce, 'should call upgrade');
+        });
+        it('logs success message after upgrade succeeds', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '99.0.0', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const upgrade = sinon_1.default.stub();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn: makeFakeSpawn(), log, upgrade });
+            strict_1.default.equal(log.callCount, 2, 'should log upgrading + upgraded messages');
+            (0, strict_1.default)(log.firstCall.args[0].includes('0.64.4'), 'should mention current version');
+            (0, strict_1.default)(log.firstCall.args[0].includes('99.0.0'), 'should mention target version');
+            (0, strict_1.default)(log.secondCall.args[0].includes('Upgraded'), 'should confirm upgrade');
+        });
+        it('falls back to notification box when upgrade fails', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '99.0.0', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const upgrade = sinon_1.default.stub().throws(new Error('npm install failed'));
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn: makeFakeSpawn(), log, upgrade });
+            // Upgrading message, failure reason, then fallback notification
+            strict_1.default.equal(log.callCount, 3, 'should log upgrading + failure reason + fallback notification');
+            (0, strict_1.default)(log.firstCall.args[0].includes('Upgrading'), 'first message should be upgrading');
+            const failMsg = (0, updateNotifier_1.stripAnsi)(log.secondCall.args[0]);
+            (0, strict_1.default)(failMsg.includes('Auto-upgrade failed'), 'should explain the failure');
+            (0, strict_1.default)(failMsg.includes('npm install failed'), 'should include error reason');
+            const plain = (0, updateNotifier_1.stripAnsi)(log.thirdCall.args[0]);
+            (0, strict_1.default)(plain.includes('99.0.0'), 'fallback should show the new version');
+            (0, strict_1.default)(plain.includes('integration-cli upgrade'), 'fallback should show manual upgrade command');
+        });
+        it('does not call upgrade when version is current', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '0.64.4', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const upgrade = sinon_1.default.stub();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn: makeFakeSpawn(), log, upgrade });
+            strict_1.default.equal(upgrade.called, false, 'should not call upgrade');
+            strict_1.default.equal(log.called, false, 'should not log anything');
+        });
+        it('does not upgrade or notify when current is newer than cached', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '0.64.4', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const upgrade = sinon_1.default.stub();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('1.0.0', { cacheFile, spawn: makeFakeSpawn(), log, upgrade });
+            strict_1.default.equal(upgrade.called, false, 'should not call upgrade');
+            strict_1.default.equal(log.called, false, 'should not log anything');
+        });
+        it('upgrades and spawns background check when cache has newer version but is stale', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const staleCache = { latest: '99.0.0', checkedAt: Date.now() - 48 * 60 * 60 * 1000 };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(staleCache));
+            const upgrade = sinon_1.default.stub();
+            const spawn = makeFakeSpawn();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log, upgrade });
+            (0, strict_1.default)(upgrade.calledOnce, 'should attempt upgrade');
+            (0, strict_1.default)(spawn.calledOnce, 'should also spawn background check since cache is stale');
+        });
+        it('spawns background check when cache is stale', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const staleCache = { latest: '0.64.4', checkedAt: Date.now() - 48 * 60 * 60 * 1000 };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(staleCache));
+            const spawn = makeFakeSpawn();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log: sinon_1.default.stub() });
+            (0, strict_1.default)(spawn.calledOnce, 'should spawn background check');
+        });
+        it('spawns background check when no cache exists', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'nonexistent.json');
+            const spawn = makeFakeSpawn();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log: sinon_1.default.stub() });
+            (0, strict_1.default)(spawn.calledOnce, 'should spawn background check');
+        });
+        it('does not spawn background check when cache is fresh', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '0.64.4', checkedAt: Date.now() };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            const spawn = makeFakeSpawn();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log: sinon_1.default.stub() });
+            strict_1.default.equal(spawn.called, false, 'should not spawn background check');
+        });
+        it('skips entirely during upgrade command', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '99.0.0', checkedAt: Date.now() - 48 * 60 * 60 * 1000 };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            process.argv = ['node', 'integration-cli', 'upgrade'];
+            const spawn = makeFakeSpawn();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log });
+            strict_1.default.equal(log.called, false, 'should not display notification');
+            strict_1.default.equal(spawn.called, false, 'should not spawn background check');
+        });
+        it('does not throw even if spawn fails', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'nonexistent.json');
+            const spawn = sinon_1.default.stub().throws(new Error('spawn failed'));
+            strict_1.default.doesNotThrow(() => (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log: sinon_1.default.stub() }));
+        });
+        it('skips when CI env var is set', () => {
+            const cacheFile = path_1.default.join(tmpDir, 'cache.json');
+            const freshCache = { latest: '99.0.0', checkedAt: Date.now() - 48 * 60 * 60 * 1000 };
+            fs_1.default.writeFileSync(cacheFile, JSON.stringify(freshCache));
+            process.env.CI = 'true';
+            const spawn = makeFakeSpawn();
+            const log = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.64.4', { cacheFile, spawn, log });
+            strict_1.default.equal(log.called, false, 'should not display notification');
+            strict_1.default.equal(spawn.called, false, 'should not spawn background check');
+        });
+    });
+    describe('integration: background check writes cache from npm', () => {
+        let tmpDir;
+        let savedCI;
+        beforeEach(() => {
+            tmpDir = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'update-notifier-integration-'));
+            savedCI = process.env.CI;
+        });
+        afterEach(() => {
+            fs_1.default.rmSync(tmpDir, { recursive: true });
+            if (savedCI !== undefined) {
+                process.env.CI = savedCI;
+            }
+            else {
+                delete process.env.CI;
+            }
+        });
+        function waitForFile(filePath, timeoutMs) {
+            const start = Date.now();
+            return new Promise((resolve, reject) => {
+                const check = () => {
+                    if (fs_1.default.existsSync(filePath))
+                        return resolve();
+                    if (Date.now() - start > timeoutMs)
+                        return reject(new Error(`Timed out waiting for ${filePath}`));
+                    setTimeout(check, 200);
+                };
+                check();
+            });
+        }
+        it('spawns a real process that fetches from npm and writes valid cache', async function () {
+            this.timeout(15000);
+            const cacheFile = path_1.default.join(tmpDir, 'subcache', 'update-check.json');
+            (0, updateNotifier_1.spawnBackgroundCheck)({
+                cacheFile,
+                spawn: child_process_1.default.spawn,
+                log: sinon_1.default.stub(),
+                upgrade: sinon_1.default.stub(),
+            });
+            await waitForFile(cacheFile, 12000);
+            const cache = JSON.parse(fs_1.default.readFileSync(cacheFile, 'utf8'));
+            (0, strict_1.default)(cache.latest, 'cache should have a latest version');
+            (0, strict_1.default)(/^\d+\.\d+\.\d+/.test(cache.latest), `latest should be semver, got: ${cache.latest}`);
+            (0, strict_1.default)(typeof cache.checkedAt === 'number', 'cache should have a checkedAt timestamp');
+            (0, strict_1.default)(cache.checkedAt <= Date.now(), 'checkedAt should not be in the future');
+        });
+        it('end-to-end: stale cache triggers spawn, fresh cache triggers upgrade', async function () {
+            this.timeout(15000);
+            const cacheFile = path_1.default.join(tmpDir, 'e2e', 'update-check.json');
+            // Unset CI so checkForUpdate runs
+            delete process.env.CI;
+            // Step 1: no cache exists — checkForUpdate should spawn a real background check
+            const upgrade1 = sinon_1.default.stub();
+            const log1 = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.0.1', { cacheFile, spawn: child_process_1.default.spawn, log: log1, upgrade: upgrade1 });
+            strict_1.default.equal(upgrade1.called, false, 'no upgrade without cache');
+            // Step 2: wait for the spawned process to write the cache
+            await waitForFile(cacheFile, 12000);
+            const cache = JSON.parse(fs_1.default.readFileSync(cacheFile, 'utf8'));
+            (0, strict_1.default)(/^\d+\.\d+\.\d+/.test(cache.latest), 'cache should contain a semver version');
+            // Step 3: run checkForUpdate again with a very old version — should trigger upgrade
+            const upgrade2 = sinon_1.default.stub();
+            const log2 = sinon_1.default.stub();
+            (0, updateNotifier_1.checkForUpdate)('0.0.1', { cacheFile, spawn: makeFakeSpawn(), log: log2, upgrade: upgrade2 });
+            (0, strict_1.default)(upgrade2.calledOnce, 'should call upgrade on second run');
+            (0, strict_1.default)(log2.firstCall.args[0].includes(cache.latest), 'should mention the target version');
+        });
+    });
+});

package/dist/test/services/oauth2.test.js

@@ -59,19 +59,38 @@ describe('OAuth2Helper', () => {
             it('opens the authorization URL', async () => {
                 await oauth2Helper.authorize();
                 sinon_1.default.assert.calledOnce(openSpy);
-                sinon_1.default.assert.calledWith(openSpy, 'https://provider.com/oauth/authorize?client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback-cli');
+                sinon_1.default.assert.calledWith(openSpy, 'https://provider.com/oauth/authorize?client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback');
             });
             it('maintains the pre-existing query parameters in authorization URL', async () => {
                 oauth2Helper['providerAuthorizationUrl'] = 'https://provider.com/oauth/authorize?query1=value1&query2=value2';
                 await oauth2Helper.authorize();
                 sinon_1.default.assert.calledOnce(openSpy);
-                sinon_1.default.assert.calledWith(openSpy, 'https://provider.com/oauth/authorize?query1=value1&query2=value2&client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback-cli');
+                sinon_1.default.assert.calledWith(openSpy, 'https://provider.com/oauth/authorize?query1=value1&query2=value2&client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback');
             });
             it('supports dynamic params', async () => {
                 oauth2Helper['providerAuthorizationUrl'] = 'https://pro-{+foo}-der.com/oauth/authorize?query1={+bar}';
                 await oauth2Helper.authorize();
                 sinon_1.default.assert.calledOnce(openSpy);
-                sinon_1.default.assert.calledWith(openSpy, 'https://pro-fooValue-der.com/oauth/authorize?query1=barValue&client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback-cli');
+                sinon_1.default.assert.calledWith(openSpy, 'https://pro-fooValue-der.com/oauth/authorize?query1=barValue&client_id=your-client-id&scope=scope1+scope2&state=eyJjbGlDYWxsYmFja1VybCI6Ii9vYXV0aDIvY2FsbGJhY2sifQ%3D%3D&response_type=code&redirect_uri=https%3A%2F%2Fintegrations-platform.unito.io%2Fcredentials%2Fnew%2Foauth2%2Fcallback');
+            });
+            describe('with legacyRedirectUrl', () => {
+                it('uses legacyRedirectUrl as redirect_uri and adds redirectToIPS to state', async () => {
+                    oauth2Helper['legacyRedirectUrl'] = 'http://localhost:3000/api/auth/notion/callback';
+                    await oauth2Helper.authorize();
+                    sinon_1.default.assert.calledOnce(openSpy);
+                    const url = new URL(openSpy.getCall(0).args[0]);
+                    strict_1.default.equal(url.searchParams.get('redirect_uri'), 'http://localhost:3000/api/auth/notion/callback');
+                    const state = JSON.parse(Buffer.from(url.searchParams.get('state'), 'base64').toString('utf-8'));
+                    strict_1.default.equal(state.redirectToIPS, 'true');
+                    strict_1.default.equal(state.cliCallbackUrl, '/oauth2/callback');
+                });
+                it('does not add redirectToIPS to state when legacyRedirectUrl is not set', async () => {
+                    await oauth2Helper.authorize();
+                    const url = new URL(openSpy.getCall(0).args[0]);
+                    const state = JSON.parse(Buffer.from(url.searchParams.get('state'), 'base64').toString('utf-8'));
+                    strict_1.default.equal(state.redirectToIPS, undefined);
+                    strict_1.default.equal(url.searchParams.get('redirect_uri'), 'https://integrations-platform.unito.io/credentials/new/oauth2/callback');
+                });
             });
         });
         describe('handleCallback', () => {
@@ -299,6 +318,27 @@ describe('OAuth2Helper', () => {
                 sinon_1.default.assert.calledOnce(res.send);
                 sinon_1.default.assert.calledWith(res.send, oauth2Namespace.HTML_ERROR_MSG('Bad request'));
             });
+            describe('with legacyRedirectUrl', () => {
+                it('uses legacyRedirectUrl as redirect_uri in token request', async () => {
+                    oauth2Helper['legacyRedirectUrl'] = 'http://localhost:3000/api/auth/notion/callback';
+                    const code = 'test-code';
+                    const req = { query: { code } };
+                    const res = { send: sinon_1.default.stub(), setHeader: sinon_1.default.stub() };
+                    await oauth2Helper['handleCallback'](req, res);
+                    const requestBody = fetchStub.getCall(0).args[1].body;
+                    const bodyParams = new URLSearchParams(requestBody);
+                    strict_1.default.equal(bodyParams.get('redirect_uri'), 'http://localhost:3000/api/auth/notion/callback');
+                });
+                it('uses standard redirect_uri when legacyRedirectUrl is not set', async () => {
+                    const code = 'test-code';
+                    const req = { query: { code } };
+                    const res = { send: sinon_1.default.stub(), setHeader: sinon_1.default.stub() };
+                    await oauth2Helper['handleCallback'](req, res);
+                    const requestBody = fetchStub.getCall(0).args[1].body;
+                    const bodyParams = new URLSearchParams(requestBody);
+                    strict_1.default.equal(bodyParams.get('redirect_uri'), 'https://integrations-platform.unito.io/credentials/new/oauth2/callback');
+                });
+            });
         });
         describe('updateToken', () => {
             const setupSuccessfulRefreshResponse = () => {

package/dist/tsconfig.tsbuildinfo

@@ -1 +1 @@
-{"root":["../src/baseCommand.ts","../src/configurationTypes.ts","../src/errors.ts","../src/index.ts","../src/commands/activity.ts","../src/commands/dev.ts","../src/commands/encrypt.ts","../src/commands/init.ts","../src/commands/invite.ts","../src/commands/login.ts","../src/commands/oauth2.ts","../src/commands/publish.ts","../src/commands/test.ts","../src/commands/upgrade.ts","../src/hooks/init/displayLogo.ts","../src/resources/configuration.ts","../src/resources/credentials.ts","../src/resources/decryption.ts","../src/resources/fileSystem.ts","../src/resources/globalConfiguration.ts","../src/resources/integrations.ts","../src/resources/integrationsPlatform.ts","../src/resources/oauth2.ts","../src/resources/template.ts","../src/services/integrationsPlatform.ts","../src/services/integrationsPlatformClient.ts","../src/services/oauth2.ts","../test/errors.test.ts","../test/commands/activity.test.ts","../test/commands/dev.test.ts","../test/commands/encrypt.test.ts","../test/commands/init.test.ts","../test/commands/invite.test.ts","../test/commands/login.test.ts","../test/commands/oauth2.test.ts","../test/commands/publish.test.ts","../test/commands/test.test.ts","../test/commands/upgrade.test.ts","../test/helpers/init.js","../test/helpers/integrations.ts","../test/helpers/styles.ts","../test/resources/configuration.test.ts","../test/resources/decryption.test.ts","../test/resources/globalConfiguration.test.ts","../test/resources/integrations.test.ts","../test/resources/oauth2.test.ts","../test/resources/template.test.ts","../test/services/integrationsPlatform.test.ts","../test/services/oauth2.test.ts","../scripts/generateTypes.ts","../.eslintrc.js"],"version":"5.9.3"}
+{"root":["../src/baseCommand.ts","../src/configurationTypes.ts","../src/errors.ts","../src/index.ts","../src/commands/activity.ts","../src/commands/dev.ts","../src/commands/encrypt.ts","../src/commands/init.ts","../src/commands/invite.ts","../src/commands/login.ts","../src/commands/oauth2.ts","../src/commands/publish.ts","../src/commands/test.ts","../src/commands/upgrade.ts","../src/hooks/init/displayLogo.ts","../src/hooks/init/updateNotifier.ts","../src/resources/configuration.ts","../src/resources/credentials.ts","../src/resources/decryption.ts","../src/resources/fileSystem.ts","../src/resources/globalConfiguration.ts","../src/resources/integrations.ts","../src/resources/integrationsPlatform.ts","../src/resources/oauth2.ts","../src/resources/template.ts","../src/services/integrationsPlatform.ts","../src/services/integrationsPlatformClient.ts","../src/services/oauth2.ts","../test/errors.test.ts","../test/commands/activity.test.ts","../test/commands/dev.test.ts","../test/commands/encrypt.test.ts","../test/commands/init.test.ts","../test/commands/invite.test.ts","../test/commands/login.test.ts","../test/commands/oauth2.test.ts","../test/commands/publish.test.ts","../test/commands/test.test.ts","../test/commands/upgrade.test.ts","../test/helpers/init.js","../test/helpers/integrations.ts","../test/helpers/styles.ts","../test/hooks/updateNotifier.test.ts","../test/resources/configuration.test.ts","../test/resources/decryption.test.ts","../test/resources/globalConfiguration.test.ts","../test/resources/integrations.test.ts","../test/resources/oauth2.test.ts","../test/resources/template.test.ts","../test/services/integrationsPlatform.test.ts","../test/services/oauth2.test.ts","../scripts/generateTypes.ts","../.eslintrc.js"],"version":"5.9.3"}

package/oclif.manifest.json

@@ -748,5 +748,5 @@
       ]
     }
   },
-  "version": "0.64.3"
+  "version": "0.64.5"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unito/integration-cli",
-  "version": "0.64.3",
+  "version": "0.64.5",
   "description": "Integration CLI",
   "bin": {
     "integration-cli": "./bin/run"